Skip to content

localData management applied - #1

Open
ttiras wants to merge 1 commit into
HamedBahram:mainfrom
ttiras:feature/localData
Open

localData management applied#1
ttiras wants to merge 1 commit into
HamedBahram:mainfrom
ttiras:feature/localData

Conversation

@ttiras

@ttiras ttiras commented Oct 31, 2023

Copy link
Copy Markdown

No description provided.

@HamedBahram

Copy link
Copy Markdown
Owner

Thanks for the PR.
I would prefer to:

  1. Read from the local storage when initializing the form and the default values instead of resetting the form in useEffect.
  2. Use the useLocalStorage from react-use instead.

This is just feedback, don't worry if you prefer not to update the PR. I'll close it.

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.

2 participants