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

[BUGFIX] Octane: should not need to use get with model.id #6829

Merged
merged 13 commits into from
Dec 2, 2019

Conversation

Gaurav0
Copy link
Contributor

@Gaurav0 Gaurav0 commented Nov 30, 2019

For #6143

Edit: Now includes a WIP bugfix, but it has TypeScript errors I don't know how to fix.

@Gaurav0 Gaurav0 changed the title Failing test for Octane model.id bug [BUGFIX] Octane: should not need to use get with model.id Dec 2, 2019
@runspired runspired requested a review from pzuraq December 2, 2019 00:52
@runspired runspired added Octane 🏷️ bug This PR primarily fixes a reported issue labels Dec 2, 2019
@runspired runspired merged commit a0158de into emberjs:master Dec 2, 2019
@runspired runspired added 🎯 beta PR should be backported to beta target:lts-3-12 🎯 release PR should be backported to release labels Dec 2, 2019
@runspired
Copy link
Contributor

Assigned Igor for backporting.

@Gaurav0 Gaurav0 deleted the octane_id_bug branch December 2, 2019 03:04
@igorT igorT removed 🎯 release PR should be backported to release 🎯 beta PR should be backported to beta labels Dec 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bug This PR primarily fixes a reported issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants