Skip to content

fix shouldComponentUpdate getting called on setState after forc…#2258

Merged
marvinhagemeister merged 1 commit into
preactjs:masterfrom
laino:fix-force-update-set-state
Jan 17, 2020
Merged

fix shouldComponentUpdate getting called on setState after forc…#2258
marvinhagemeister merged 1 commit into
preactjs:masterfrom
laino:fix-force-update-set-state

Conversation

@laino

@laino laino commented Jan 17, 2020

Copy link
Copy Markdown
Contributor

See the added test.

_force also had a weird type.

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.07%) to 97.745% when pulling 97af3a2 on laino:fix-force-update-set-state into 6771b23 on preactjs:master.

@JoviDeCroock JoviDeCroock left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I like this, this also shaves some bytes! Thank you for that!

@marvinhagemeister marvinhagemeister left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sweet! This is super awesome 👍 Thank you so much for the PR ❤️

@marvinhagemeister marvinhagemeister changed the title fix shouldComponentUpdate getting called on setState after forceUpdate fix shouldComponentUpdate getting called on setState after forc… Jan 17, 2020
@marvinhagemeister marvinhagemeister merged commit e3591cc into preactjs:master Jan 17, 2020
@pika-ci

pika-ci Bot commented Jan 17, 2020

Copy link
Copy Markdown

🚀 This PR has been merged! Once a new release is created, any changes will become available on npm. Until then, you can load and install it directly from the Pika CDN:

npm install https://cdn.pika.dev/preact/master

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.

4 participants