Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable Ubuntu 14.04 beta in travis #8

Open
wants to merge 40 commits into
base: master
Choose a base branch
from
Open

Conversation

dastergon
Copy link
Contributor

Travis' team recently announced that they opened Ubuntu 14.04 as beta.

Since PR #6 is relevant to Ubuntu 14.04 compatibility, I enabled Ubuntu Trusty beta to run tests to its proper environment.

At beta stage, travis-ci does not include pre-installed PHP environment so I am currently installing php from install scripts

Contains PR #6

Replaced all `authorizationKey` checks with a listener that performs the
same check at a single point, right after a request is reveived.
Off-loading such checks from the controllers keeps the code DRY
and understandable.
- Added comments describing what the method does.
- Improved the method code
Updated the method that handles the response with the contents of the
JStree:
- Used early-return if's
- Moved the code that updates the tree to a dedicated method.
fpapadopou and others added 9 commits November 5, 2015 13:41
In some cases there are no route parameters in the request and this
should be handled in the AuthListener of the application, otherwise an
exception will be raised
At the beta stage, travis-ci does not include preinstalled PHP environment
so I am currently installing php from install scripts
@spiliot
Copy link
Contributor

spiliot commented Jan 31, 2016

@dastergon Could we check if this is solved now from Travis' side?

@dastergon
Copy link
Contributor Author

@spiliot the mysql issue is not yet resolved on their side and it won't be in the near future.
We have to update travis' install scripts to install and manage our own mysql server.
Let me know if you have any other questions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants