ember-fastboot / ember-cli-fastboot

Server-side rendering for Ember.js apps
http://ember-fastboot.com/
MIT License
851 stars 159 forks source link

fix: Provide a more meaningful error when whitelist entry is missing #903

Closed runspired closed 1 year ago

runspired commented 1 year ago

In emberjs/data#8101 it was noted that the error message provided here did not give enough context for users to understand the issue was due to the Fastboot environment and the existence of the fastbootDependencies allow list. This should help greatly with that.

Ref: https://github.com/emberjs/data/issues/8101#issuecomment-1341675294

runspired commented 1 year ago

cc @elwayman02

elwayman02 commented 1 year ago

@ef4 thanks for fixing the prettier formatting. It looks like all the tests are passing now - if this change is desirable as-written, would this be good to merge?