Skip to content

Conversation

@rajakvk
Copy link
Contributor

@rajakvk rajakvk commented Oct 1, 2021

Closes item-1 at #775

@locks locks requested a review from jenweber October 14, 2021 11:53
@rajakvk rajakvk mentioned this pull request Oct 15, 2021

init() {
super.init(...arguments);
constructor(ids, args) {
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
constructor(ids, args) {
constructor() {

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Incorporated review comments with commit 662f8d6

init() {
super.init(...arguments);
constructor(ids, args) {
super(ids, args);
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
super(ids, args);
super(...arguments);

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Incorporated review comments with commit 662f8d6

@rajakvk rajakvk mentioned this pull request Oct 18, 2021
@jenweber jenweber merged commit f8ab77c into ember-learn:master Oct 21, 2021
@jenweber
Copy link
Contributor

Thank you so much for your patience waiting for reviews. I will try to do better from here on out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants