-
Notifications
You must be signed in to change notification settings - Fork 0
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
📩 Store comment draft in local storage per submission #453
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @ClumsyVlad and the rest of your teammates on Graphite |
2f54c10
to
5c8d6de
Compare
48b142f
to
ffedbff
Compare
src/lib/localStorage.ts
Outdated
@@ -3,6 +3,7 @@ import { z } from "zod"; | |||
export const LOCAL_STORAGE_KEYS = { | |||
waveStepsData: "waveStepsData", | |||
applicationStepsData: "applicationStepsData", | |||
commentsData: "commentsData", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It needs at least application id
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, but not in this array I believe 🤔
a4f9cc2
to
d9cb2d9
Compare
ffedbff
to
1b27d2e
Compare
src/lib/localStorage.ts
Outdated
@@ -3,9 +3,10 @@ import { z } from "zod"; | |||
export const LOCAL_STORAGE_KEYS = { | |||
waveStepsData: "waveStepsData", | |||
applicationStepsData: "applicationStepsData", | |||
commentData: "commentData", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
commentData: "commentData", | |
commentData: (applicationId) => `commentData-${applicationId}`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure how types in getLocalStorageValue
should work then 🤔
d9cb2d9
to
1b2346d
Compare
1b27d2e
to
3e03fde
Compare
c4ac13e
to
56a396d
Compare
▲ Vercel deployment successful! Preview Link |
No description provided.