Skip to content

Conversation

@AmauryD
Copy link
Contributor

@AmauryD AmauryD commented Dec 20, 2023

Fixes #1507

classes;

@hasMany('class', { async: false, inverse: null })
@hasMany('class', { async: false, inverse: null, polymorphic: true })
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If someone knowledgeable about this library comes across this, is it normal for 'components' to be a 'class' model rather than a 'component' model ?

Copy link
Collaborator

@SergeAstapov SergeAstapov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

Copy link
Member

@RobbieTheWagner RobbieTheWagner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems reasonable to me, thanks for the fix!

@RobbieTheWagner RobbieTheWagner merged commit 191d51b into ember-learn:master Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error from ember data with the module model

3 participants