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

WIP: Upgrade dependencies #893

Closed
wants to merge 1 commit into from

Conversation

bjornharrtell
Copy link
Contributor

@bjornharrtell bjornharrtell commented Jul 19, 2017

Just can't stop upgrading... known blocking issues to be resolved:

  • Assertion Failed: You attempted to define a {{link-to "category"}} but did not pass the parameters required for generating its dynamic segments. Assertion Failed: Attempting to inject an unknown injection: 'controller:category'
  • Assertion Failed: You modified "crate.owner_team" twice on <cargo@model:crate::ember584:libc> in a single render. It was rendered in "template:cargo/templates/crate/version.hbs" and modified in "template:cargo/templates/crate/version.hbs". This was unreliable and slow in Ember 1.x and is no longer supported. See [Glimmer 2] "Backtracking re-render" is now an assertion emberjs/ember.js#13948 for more details.
  • Assertion Failed: You modified "crate.keywords.isPending" twice on DS.PromiseManyArray:ember585 in a single render. It was rendered in "template:cargo/templates/crate/version.hbs" and modified in "template:cargo/templates/crate/version.hbs". This was unreliable and slow in Ember 1.x and is no longer supported. See [Glimmer 2] "Backtracking re-render" is now an assertion emberjs/ember.js#13948 for more details.
  • TypeError: Cannot read property 'destroy' of undefined
    at Class.willDestroyElement (copy-button.js:50)

Also, ember-data was downgraded at some point from 2.13 to 2.12. Not sure what the issue was or if it might be resolved in 2.14.

@mmun
Copy link

mmun commented Aug 10, 2017

@bjornharrtell You may want to try this new version of ember data which may fix those two "modified twice" errors.

Unfortunately, I think the other two errors are unrelated :P

@Turbo87
Copy link
Member

Turbo87 commented Nov 30, 2017

Closing due to inactivity and conflicts. Ember itself has been upgraded in #1180.

@Turbo87 Turbo87 closed this Nov 30, 2017
@bjornharrtell bjornharrtell deleted the deps-upgrade branch November 30, 2017 15:15
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 this pull request may close these issues.

4 participants