-
Notifications
You must be signed in to change notification settings - Fork 33
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
Materia v5.0.0: Ifrit #1204
Merged
Merged
Materia v5.0.0: Ifrit #1204
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ered list pretending to be a table.
adds a new docker script run_build_release_package.sh creates a packaged zip file with the minimum required deps creates a build-info.yml file contianing hashes and build info adds release option to travis to upload the zips to the github releases when tagged
Materia server's urls to the guide pages should be standardized EG: id-clean_name/players-guidem, id-clean_name/creators-guide So this chagne simplifies those urls. The result is no angular controller is needed to display the guide pages. The server knows exactly what each guide page url is going to display There's an issue where widgets without guide pages break, fixed here. relocated/styled the links a bit - still needs some more changed routes so it doesn't match urls that'll never be used (only players-guide and creators-guide) started changing 'player guide' and 'creator guide' to player's guide, etc (still more to do)
update to node 12, reorganize css
* adds a .env file for docker to pull vars from * consolidates db connection env vars * adds a tag env to docker compose to allow docker images to be versioned w repo * adds docker-compose.build.yml for building releases inside a container * updates lots of docker convenience scripts * removes scripts for building containers now that docker-compose build can do the same
* Added run_create_me.sh to easily create a default user based on you * updated output in some admin and widget tasks * added ability to set a user's password via tasks * bumped docker-compose files to v 3.5 to gain network and container names * general polish on docker scripts
…-and-environ-for-deploys polish docker scripts and clean up release builds
FrenjaminBanklin
approved these changes
Jun 4, 2019
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Buttons don't seem to follow the same rules throughout the entire application but that can be resolved later.
All of the changes appear to be functioning properly.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull request is WIP.
Includes the following fixes and features:
install.yaml
. Students must share the widget with abasic_author
who can then publish the widget includes a migrationNote this PR should be implemented along side Client-Assets' 1.3.0 branch: ucfopen/Materia-Server-Client-Assets#50