-
Notifications
You must be signed in to change notification settings - Fork 161
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
Add support for custom configuration #752
Conversation
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.
Thanks for making this change!
index.js
Outdated
@@ -79,6 +79,8 @@ module.exports = { | |||
this._appRegistry = app.registry; | |||
this._name = app.name; | |||
|
|||
this.options = this._optionsFor(app.env, app.project); |
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.
Nitpick: Maybe better to name this this.fastbootOptions
for clarity.
a087a27
to
886f9ab
Compare
c84f169
to
0ecc1da
Compare
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.
Please update the README and then I can merge this.
0ecc1da
to
355a358
Compare
Done ✅, the section uses option and link for |
355a358
to
ad4690f
Compare
This is good to go IMO after the minor update to readme about |
ad4690f
to
40b6634
Compare
Thanks for addressing the comments. Merging this. |
This adds support for reading FastBoot configuration from
config/fastboot.js
(similar to how other add-ons likeember-intl
works).It should be possible to share this file with production server
fastboot-app-server
.TODOs
README.md