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

feat:added localstorage #60

Open
wants to merge 1 commit into
base: staging
Choose a base branch
from
Open

feat:added localstorage #60

wants to merge 1 commit into from

Conversation

dhruvsol
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Sep 18, 2022

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

Name Status Preview Updated
metapass-v2 ✅ Ready (Inspect) Visit Preview Sep 18, 2022 at 4:06PM (UTC)

@vercel vercel bot temporarily deployed to Preview – metapass-v2 September 18, 2022 16:00 Inactive
@anoushk1234 anoushk1234 requested a review from dawksh September 23, 2022 14:22
@dawksh
Copy link
Member

dawksh commented Sep 23, 2022

can someone tell me why are we storing localstorage vars?

@dhruvsol
Copy link
Contributor Author

can someone tell me why are we storing localstorage vars?

so if somethings mess up during the event creation so then the user doesn't have to rewrite the whole thing again we can send him back to form 1 (step 1 ) and he will have data in local storage there he can check the data and the progress remain the same

and don't merge it , it has a lot of type issue

@anoushk1234
Copy link
Collaborator

@dhruvsol can you push this to staging first?

@dawksh
Copy link
Member

dawksh commented Oct 12, 2022

what is the status of this?

@anoushk1234
Copy link
Collaborator

what is the status of this?

Conflicts have to be resolved then can push to staging

@anoushk1234 anoushk1234 changed the base branch from prod to staging October 24, 2022 08:05
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.

3 participants