Skip to content
This repository was archived by the owner on Aug 17, 2025. It is now read-only.

add flexibility to componentDidUpdate#250

Open
donaldsin wants to merge 1 commit into
arfedulov:masterfrom
donaldsin:master
Open

add flexibility to componentDidUpdate#250
donaldsin wants to merge 1 commit into
arfedulov:masterfrom
donaldsin:master

Conversation

@donaldsin

Copy link
Copy Markdown

No description provided.

@donaldsin

donaldsin commented Apr 11, 2024

Copy link
Copy Markdown
Author

imagine when there is a state change to clear the current value (i.e. the value is an empty String), it will be better to reset the default opening screen to current date, instead of caching the previous value.
Plus, It will throw error when there is a change in maxDate or minDate when value is empty, as the state of date, month and year were cached, and it is out of range

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.

1 participant