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

Allow extra configuration file #259

Merged
merged 3 commits into from
Aug 2, 2017
Merged

Allow extra configuration file #259

merged 3 commits into from
Aug 2, 2017

Conversation

multani
Copy link
Contributor

@multani multani commented Jul 31, 2017

In order to override some configuration settings, we can now specify an
additional configuration location using the Java system properties
ice.config.location which points to another configuration file
location.

Otherwise, the only way to set custom configuration parameters is by
overriding this while Config.groovy file.

In order to override some configuration settings, we can now specify an
additional configuration location using the Java system properties
``ice.config.location`` which points to another configuration file
location.

Otherwise, the only way to set custom configuration parameters is by
overriding this while Config.groovy file.
NitriKx
NitriKx previously approved these changes Aug 1, 2017
@NitriKx NitriKx self-assigned this Aug 1, 2017
@NitriKx NitriKx added this to the v1.1.2 milestone Aug 1, 2017
@NitriKx
Copy link
Member

NitriKx commented Aug 1, 2017

Thanks for contributing! Can you update the README with a description of the new parameter?

Copy link
Member

@NitriKx NitriKx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

README update is needed

@NitriKx NitriKx dismissed their stale review August 1, 2017 07:44

README update is needed

@multani
Copy link
Contributor Author

multani commented Aug 1, 2017

@NitriKx That should be better now :)

README.md Outdated

If you need to setup custom Grails settings, you can specify an additional configuration file to be loaded by Grails by setting the ``ice.config.location`` system property to the location of that file.

See http://docs.grails.org/2.3.7/guide/single.html#configExternalized for more information.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be replaced with http://docs.grails.org/2.4.4/guide/single.html#configExternalized (as we use 2.4.4 in Ice)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, sorry I missed that!

Copy link
Member

@NitriKx NitriKx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect! Thanks

@NitriKx NitriKx merged commit e146c28 into Teevity:master Aug 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants