fix: add locked image state + qr deletion + other misc fixes#91
Conversation
|
Visit the preview URL for this PR (updated for commit 2ab4702): https://dev-nwplus-admin--pr91-fix-stampbook-x8uhu9k8.web.app (expires Fri, 23 Jan 2026 09:13:15 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: c290bf9e2fac0401389f751f415cce4267517b51 |
|
|
||
| setLockedImageFile(null); | ||
| setLockedImagePreview(null); | ||
| setLockedImageError(null); |
There was a problem hiding this comment.
I lowkey just copied over image select, as of right now don't think it's worth refactoring to its own component
|
functionality lgtm i think tyyyy! one thing i wanna confirm is it looks like on stamp creation only the stamp image is required but the locked image is optional. is it the case that if no one uploads anything there it would default to the locked one we have now?? just wanted to make sure it doesnt error / glitch out or anything if that happens on portal side |
Yep on portal it will default to the one we have rn, kept it as optional for now as we backfill the stamps. Until I merge this PR on portal it won't use the locked stamps image put I can push that change to dev separately if needed: nwplus/portal-v2#42 |
Reason
isQRUnlockabletoggled off