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

Omit selected fields from being persisted #6

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Ciaran0
Copy link

@Ciaran0 Ciaran0 commented Feb 26, 2018

Small PR to give Formik-Persist the ability to ignore / omit certain fields from being persisted to local storage.

There is a number of real world use cases where people might want to do this. For example password confirmation, email confirmation etc.

Let me know what you think.

Really enjoying Formik!
Cheers.

@Ciaran0
Copy link
Author

Ciaran0 commented Feb 27, 2018

Just realised I should ignore touched and errors for those fields. Ill add this

@Ciaran0
Copy link
Author

Ciaran0 commented Mar 5, 2018

@jaredpalmer What do you think of this?

@dzienisz
Copy link

@Ciaran0 can you resolve conflicts?

@Ciaran0
Copy link
Author

Ciaran0 commented Dec 1, 2018

Hi @dzienisz,
I would but I don't see this being merged in anytime soon

…into ignore-selected-fields

# Conflicts:
#	package.json
#	src/formik-persist.tsx
#	test/formik-persist.test.tsx
@Ciaran0
Copy link
Author

Ciaran0 commented May 18, 2019

@jaredpalmer Thanks for approving this 😄
I have resolved the conflicts.

@quocb
Copy link

quocb commented Aug 29, 2019

This is useful. Is this going to be merged?

@hendrul
Copy link

hendrul commented Sep 2, 2019

I need it to ignore isSubmitting... merge when??

@quocb
Copy link

quocb commented Sep 2, 2019

^^ that and need to ignore Password fields too. If this doesn't come out soon I'll just have to not use this and roll my own.

@hendrul
Copy link

hendrul commented Sep 6, 2019

Men this only works for values, touched, and errors but not for other fields like isSubmitting

@rodgomesc
Copy link

why this is not merged yeat????? cmonnn

@hendrul
Copy link

hendrul commented Oct 15, 2019

why this is not merged yeat????? cmonnn

hahahaha Que mal inglessss! xDDD

@mhilland
Copy link

+1

@dzienisz
Copy link

dzienisz commented Nov 1, 2019

Yes!!!

@mwmcode
Copy link

mwmcode commented Dec 18, 2019

Would this work with nested fields <Field name='address.city' />?

@Romusc
Copy link

Romusc commented Jul 28, 2020

@jaredpalmer Pleeeeeaaase merge !

@ma-v
Copy link

ma-v commented Dec 16, 2020

Up @jaredpalmer, pleaase ! 🙏
Many thanks

@jayzyaj
Copy link

jayzyaj commented Dec 21, 2020

Up @jaredpalmer!!

@jayzyaj
Copy link

jayzyaj commented Jan 26, 2021

Any updates? @jaredpalmer

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.

None yet