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

incompatible with ember-cli-app-version #47

Closed
basz opened this issue Jun 27, 2022 · 7 comments
Closed

incompatible with ember-cli-app-version #47

basz opened this issue Jun 27, 2022 · 7 comments

Comments

@basz
Copy link

basz commented Jun 27, 2022

ember-cli-app-version changes the environment object; APP['version'] is set.

ember-get-config reads it before that change I think.

so in an app this works;

import config from '../config/environment';
console.log(config.APP['version']);

while this does not;

import config from 'ember-get-config';
console.log(config.APP['version']);

oooor,... i could be doing it wrong...

@bertdeblock
Copy link
Contributor

This would be solved by #45.

@mansona
Copy link
Owner

mansona commented Jun 28, 2022

@mansona mansona closed this as completed Jun 28, 2022
@basz
Copy link
Author

basz commented Jun 29, 2022

hi, i've tested a bit more.

When using embroider this still happens

@bertdeblock
Copy link
Contributor

@basz That's because under Embroider, a compat adapter is used instead of ember-get-config itself.
We should PR that the compat adapter is not used when embet-get-config >= v2.1 is detected I think.

@bertdeblock
Copy link
Contributor

Forgot to mention, you can also disable the compat adapter in your project for now.

@bertdeblock
Copy link
Contributor

Made embroider-build/embroider#1224.

@bertdeblock
Copy link
Contributor

The adapter is now disabled in Embroider v1.8.1:
https://github.com/embroider-build/embroider/releases/tag/v1.8.1

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

No branches or pull requests

3 participants