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

3.28: Awaiting async relationships does not work anymore #7684

Closed
Turbo87 opened this issue Aug 29, 2021 · 1 comment · Fixed by #7691
Closed

3.28: Awaiting async relationships does not work anymore #7684

Turbo87 opened this issue Aug 29, 2021 · 1 comment · Fixed by #7691

Comments

@Turbo87
Copy link
Member

Turbo87 commented Aug 29, 2021

Reproduction

Description

Just like in previous Ember Data releases I expected to be able to await async relationships to be able to wait until the content has loaded before continuing, but this appears to have broken with v3.28 by resolving before the network request has resolved.

Versions

Run the following command and paste the output below: yarn list ember-source && yarn list ember-cli && yarn list --pattern ember-data.

❯ yarn list ember-source && yarn list ember-cli && yarn list --pattern ember-data
yarn list v1.22.4
warning Filtering by arguments is deprecated. Please use the pattern option instead.
└─ [email protected]
✨  Done in 1.21s.
yarn list v1.22.4
warning Filtering by arguments is deprecated. Please use the pattern option instead.
└─ [email protected]
✨  Done in 0.90s.
yarn list v1.22.4
├─ @ember-data/[email protected]
├─ @ember-data/[email protected]
├─ @ember-data/[email protected]
├─ @ember-data/[email protected]
├─ @ember-data/[email protected]
├─ @ember-data/[email protected]
├─ @ember-data/[email protected]
├─ @ember-data/[email protected]
├─ @ember-data/[email protected]
├─ [email protected]
└─ [email protected]
✨  Done in 0.90s.
@igorT
Copy link
Member

igorT commented Aug 31, 2021

Thanks for the detailed issue report, it seems like a few people have run into this, we are working on a fix in #7691

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 a pull request may close this issue.

3 participants