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

fix component name issue in ember 2.8.3 #153

Merged
merged 2 commits into from
Oct 21, 2017

Conversation

quincyle
Copy link
Contributor

Fixes #152

This project uses semver, please check the scope of this pr:

  • #none# - documentation fixes and/or test additions
  • #patch# - backwards-compatible bug fix
  • #minor# - adding functionality in a backwards-compatible manner
  • #major# - incompatible API change

CHANGELOG

  • Fixed issue when running against 2.8.3, List can't get the valid component name to render.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 0e24464 on quincyle:small-view into ** on ciena-frost:master**.

@dafortin
Copy link
Member

@jscharf I will approve, but I think we should do what we discussed. Instead of checking the component name checking we should use hard coded keys for the default conversion. This will avoid checking the component name and the different version handling. @jscharf can you do another PR on Monday to fix this? Thx

@dafortin dafortin merged commit 4086a1a into ciena-frost:master Oct 21, 2017
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.

List 6.0.0 doesn't work with ember 2.8.3
3 participants