8000 Modernize empty-state component by vladucu · Pull Request #614 · smile-io/ember-polaris · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Aug 2, 2024. It is now read-only.

Modernize empty-state component #614

Merged
merged 1 commit into from
Jun 22, 2020

Conversation

vladucu
Copy link
Member
@vladucu vladucu commented Jun 22, 2020

No description provided.

@vladucu vladucu self-assigned this Jun 22, 2020
Copy link
Contributor
@tomnez tomnez left a comment

Choose a reason for hiding this comment

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

Looks good but left a question just for my own understanding.

* @type {Object}
* @default null
*/
action: null,
action = null;
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this break in our version of Ember here? I thought we had to update the use of attributes named action at some point in our Ember upgrades but maybe I'm remembering wrong.

Copy link
Member Author

Choose a reason for hiding this comment

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

good catch.... it seems to work, but we should probably do this just to avoid confusions
have this noted to recheck once all components are done

<PolarisButton
@plain={{true}}
@text={{@secondaryAction.text}}
@ @secondaryAction.onAction}}
Copy link
Contributor

Choose a reason for hiding this comment

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

fn helper?

Copy link
Member Author

Choose a reason for hiding this comment

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

🙌

@vladucu vladucu merged commit a589f39 into refactor/es6-classes Jun 22, 2020
@vladucu vladucu deleted the refactor/es6-empty-state branch June 22, 2020 19:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0