Skip to content
This repository was archived by the owner on Jul 14, 2025. It is now read-only.

Conversation

jjaffeux
Copy link
Contributor

@jjaffeux jjaffeux commented Apr 24, 2024

This has regressed in recent changes, this commit fixes the issue and also adds a test.

@jjaffeux jjaffeux requested a review from jancernik April 24, 2024 13:48
@jjaffeux jjaffeux merged commit cbf3f3a into main Apr 24, 2024
@jjaffeux jjaffeux deleted the ctrl-enter-save branch April 24, 2024 13:54
@@ -41,7 +41,7 @@ export default class TopicAssignments extends Component {
</div>
<Assignment
@assignment={{this.selectedAssignment}}
@onSubmit={{this.submit}}
@onSubmit={{this.args.onSubmit}}
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
@onSubmit={{this.args.onSubmit}}
@onSubmit={{@onSubmit}}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ah yes shit, we don't have linting here 🙃

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants