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

updated readme with examples, addressed two real-world issues. #7

Merged
merged 1 commit into from
Dec 6, 2017

Conversation

webbtj
Copy link
Contributor

@webbtj webbtj commented Dec 6, 2017

Credential key names can be ambiguous, so they are now namespaced in Laravel with the resource's label. The config helper is not reliable inside config files, so you cannot reference a config variable defined by Manifold inside other config files. To address this, an aliases section was added to the manifold.php config file so that you can define what existing configs should be overwritten by which Manifold configs -- main use case would be for DB creds. See readme

…tial key names can be ambiguous, so they are now namespaced in Laravel with the resource's label. The config helper is not reliable inside config files, so you cannot reference a config variable defined by Manifold inside other config files. To address this, an aliases section was added to the manifold.php config file so that you can define what existing configs should be overwritten by which Manifold configs -- main use case would be for DB creds. See readme
@CLAassistant
Copy link

CLAassistant commented Dec 6, 2017

CLA assistant check
All committers have signed the CLA.

@PLaRoche PLaRoche merged commit 392d5d7 into master Dec 6, 2017
@PLaRoche PLaRoche deleted the webbtj/config-aliases branch December 18, 2017 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants