chore(config): allow svg and increase body size #1729
Conversation
purusott
force-pushed
the
file-upload
branch
from
2f22a5f
to
4186e0d
Compare
| return setFormError(getFormFeedbackForError('file/invalid')) | ||
| case 403: | ||
| return setFormError( | ||
| getFormFeedbackForError('auth/operation-not-allowed'), |
There was a problem hiding this comment.
Vil denne casen noen gang vises? Siden om man får 403, så blir man bare redirected til forsiden og denne formErroren vil aldri vises til noen
There was a problem hiding this comment.
Ja, det stemmer! Har tenkt litt over det og mtp at det er gjennom APIet tenker jeg at vi heller bør fjerne redirect fra den og bare returnere 403 (forbidden). Da får man riktig respons om man prøver å poste utenfor uiet vårt 🤔 Jeg har ikke sett et API returnere redirect om man ikke har tilgang 💭 det passer vel mer til slik vi hadde det før
There was a problem hiding this comment.
Brukere vil jo kjenne til dette endepunketved å se via nettverkskall og slikt. Burde vi sikre det på noe vis? Nå kan man pr def spamme oss ned med logoer
There was a problem hiding this comment.
bra tenkt :) Skal undersøke rate-limiting.
| setFile(e.target.value) | ||
| setFileName(e.target?.files?.item(0)?.name ?? 'Logo uten navn') | ||
|
|
||
| const selectedFile = e.target.files?.[0] |
There was a problem hiding this comment.
Kunne man heller returnert denne mye tidligere om files ikke fantes? Så slipper vi å håndtere !selctedFile?
After: can upload svgs and files with size less than 10mb (svg img uploaded below)