Skip to content
This repository has been archived by the owner on Dec 5, 2024. It is now read-only.

fix: position update #207

Merged
merged 3 commits into from
Aug 9, 2018
Merged

fix: position update #207

merged 3 commits into from
Aug 9, 2018

Conversation

FezVrasta
Copy link
Member

No description provided.

@FezVrasta FezVrasta force-pushed the fix/position-update-jquense branch from 88a3de4 to f1c1f26 Compare August 9, 2018 14:36
@FezVrasta
Copy link
Member Author

I think the new placement state is enough to trigger an updated when needed, that's why it's working without the getDerivedStateFromProps

@FezVrasta FezVrasta merged commit 86701f2 into master Aug 9, 2018
@FezVrasta FezVrasta deleted the fix/position-update-jquense branch August 9, 2018 14:40
@FezVrasta
Copy link
Member Author

Released as v1.0.1

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

Successfully merging this pull request may close these issues.

2 participants