Skip to content
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

Improve/addons hooks #9945

Closed
wants to merge 7 commits into from
Closed

Improve/addons hooks #9945

wants to merge 7 commits into from

Conversation

ndelangen
Copy link
Member

Migrate some addons to hooks

@ndelangen ndelangen self-assigned this Feb 24, 2020
@ndelangen ndelangen added the maintenance User-facing maintenance tasks label Feb 24, 2020
setState({
...(state as RanState),
status: 'ready',
});
Copy link
Contributor

Choose a reason for hiding this comment

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

why does this change from being called as the 2nd parameter of setState?

@jonspalmer
Copy link
Contributor

I like it. I've overlapped with you here #9951 on the removing of some of the deprecated cruft.

@ndelangen
Copy link
Member Author

@jonspalmer I'll close this PR in favour of yours, and you cam perhaps cherry-pick some of my changes?

@ndelangen ndelangen closed this Feb 26, 2020
@jonspalmer
Copy link
Contributor

jonspalmer commented Feb 26, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance User-facing maintenance tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants