Skip to content

Fix a deprecation warning during ember build #2088

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

Merged
merged 1 commit into from
Jan 5, 2020

Conversation

jtgeibel
Copy link
Member

@jtgeibel jtgeibel commented Jan 5, 2020

The lint references https://emberjs.com/deprecations/v3.x#toc_ember-component-send-action. There is no usage of sendAction in the component, so I believe no further changes are needed there.

r? Turbo87

There is no usage of `sendAction` in the component, so I believe no futher changes are needed there.

    DEPRECATION: Passing actions to components as strings (like `<Input @Enter="saveToken" />`) is deprecated.
    Please use closure actions instead (`<Input @Enter={{action "saveToken"}} />`).
    ('cargo/templates/components/api-token-row.hbs' @ L4:C6)  [deprecation id: ember-component.send-action]
    See https://emberjs.com/deprecations/v3.x#toc_ember-component-send-action for more details.
@rust-highfive
Copy link

r? @smarnach

(rust_highfive has picked a reviewer for you, use r? to override)

Copy link
Member

@Turbo87 Turbo87 left a comment

Choose a reason for hiding this comment

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

looks alright to me 👍

I guess ultimately we would want to wrap the input in a form and use the submit event instead of listening for the enter key in the input itself, but that can be refactored later :)

@jtgeibel
Copy link
Member Author

jtgeibel commented Jan 5, 2020

@bors r=Turbo87

@bors
Copy link
Contributor

bors commented Jan 5, 2020

📌 Commit d4798bd has been approved by Turbo87

@bors
Copy link
Contributor

bors commented Jan 5, 2020

⌛ Testing commit d4798bd with merge 941b460...

bors added a commit that referenced this pull request Jan 5, 2020
Fix a deprecation warning during `ember build`

The lint references https://emberjs.com/deprecations/v3.x#toc_ember-component-send-action.  There is no usage of `sendAction` in the component, so I believe no further changes are needed there.

r? Turbo87
@bors
Copy link
Contributor

bors commented Jan 5, 2020

☀️ Test successful - checks-travis
Approved by: Turbo87
Pushing 941b460 to master...

@bors bors merged commit d4798bd into rust-lang:master Jan 5, 2020
@jtgeibel jtgeibel deleted the ember-lint-use-closure-action branch January 6, 2020 01:36
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.

5 participants