Skip to content

Omit selected fields from being persisted#6

Open
Ciaran0 wants to merge 5 commits into
jaredpalmer:masterfrom
Ciaran0:ignore-selected-fields
Open

Omit selected fields from being persisted#6
Ciaran0 wants to merge 5 commits into
jaredpalmer:masterfrom
Ciaran0:ignore-selected-fields

Conversation

@Ciaran0

@Ciaran0 Ciaran0 commented Feb 26, 2018

Copy link
Copy Markdown

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

Ciaran0 commented Feb 27, 2018

Copy link
Copy Markdown
Author

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

@Ciaran0

Ciaran0 commented Mar 5, 2018

Copy link
Copy Markdown
Author

@jaredpalmer What do you think of this?

@dzienisz

Copy link
Copy Markdown

@Ciaran0 can you resolve conflicts?

@Ciaran0

Ciaran0 commented Dec 1, 2018

Copy link
Copy Markdown
Author

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

Ciaran0 added 3 commits May 18, 2019 18:39
…into ignore-selected-fields

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

Ciaran0 commented May 18, 2019

Copy link
Copy Markdown
Author

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

@quocb

quocb commented Aug 29, 2019

Copy link
Copy Markdown

This is useful. Is this going to be merged?

@hendrul

hendrul commented Sep 2, 2019

Copy link
Copy Markdown

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

@quocb

quocb commented Sep 2, 2019

Copy link
Copy Markdown

^^ 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

hendrul commented Sep 6, 2019

Copy link
Copy Markdown

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

@rodgomesc

Copy link
Copy Markdown

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

@hendrul

hendrul commented Oct 15, 2019

Copy link
Copy Markdown

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

hahahaha Que mal inglessss! xDDD

@mhilland

Copy link
Copy Markdown

+1

@dzienisz

dzienisz commented Nov 1, 2019

Copy link
Copy Markdown

Yes!!!

@mwmcode

mwmcode commented Dec 18, 2019

Copy link
Copy Markdown

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

@Romusc

Romusc commented Jul 28, 2020

Copy link
Copy Markdown

@jaredpalmer Pleeeeeaaase merge !

@ma-v

ma-v commented Dec 16, 2020

Copy link
Copy Markdown

Up @jaredpalmer, pleaase ! 🙏
Many thanks

@jayzyaj

jayzyaj commented Dec 21, 2020

Copy link
Copy Markdown

Up @jaredpalmer!!

@jayzyaj

jayzyaj commented Jan 26, 2021

Copy link
Copy Markdown

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.