Skip to content

wip: adding form clear on reload#11

Open
InsaneZein wants to merge 6 commits into
mainfrom
form-clear
Open

wip: adding form clear on reload#11
InsaneZein wants to merge 6 commits into
mainfrom
form-clear

Conversation

@InsaneZein

Copy link
Copy Markdown
Contributor

I'm trying to add a form clear, but I've got one little hiccup: if I click "reset", the form clears but the list fields don't go away

image

@vercel

vercel Bot commented Sep 19, 2022

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
twotop ✅ Ready (Inspect) Visit Preview Sep 22, 2022 at 3:18PM (UTC)

@ryelo

ryelo commented Sep 20, 2022

Copy link
Copy Markdown
Member

@InsaneZein does this reset the fields when moving around the site or just when you remount the page?

@InsaneZein

Copy link
Copy Markdown
Contributor Author

@InsaneZein does this reset the fields when moving around the site or just when you remount the page?

When you move around the site. It resets it whenever the page gets initialized.

@ryelo

ryelo commented Sep 22, 2022

Copy link
Copy Markdown
Member

@InsaneZein can we hold this PR until I lock the submit button behind auth. I don't want people to have to navigate away and then lose all of their recipe if they've already started it

Also, I want to make sure that if a person generates their recipe and then wants to go back to edit it, that it doesn't get removed from state

@vercel vercel Bot temporarily deployed to Preview September 22, 2022 15:18 Inactive
@InsaneZein

Copy link
Copy Markdown
Contributor Author

Also, I want to make sure that if a person generates their recipe and then wants to go back to edit it, that it doesn't get removed from state

I'm currently working on this part. I'm having trouble figuring out how to get back to the BuilderForm route with existing data.

@render

render Bot commented Sep 23, 2022

Copy link
Copy Markdown

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