app icon indicating copy to clipboard operation
app copied to clipboard

Možnosť nahrať a zmazať profilovú foto v nastaveniach profilu

Open laurabaluchova opened this issue 1 year ago • 1 comments

Čo je hotové:

  • ako výchozí je aktuálna slack foto alebo default avatar (teda bude po migrácii dát)
  • užívateľ vie nahrať novú foto (overuje sa jej veľkosť a typ súboru) a vidí preview
  • foto je možné zmazať (ostane default avatar)
  • všetky potvrdené zmeny (uloženie novej foto a zmazanie) sa ukladajú do databáze do políčka profilePictureUrl

TODO / problémy:

  • [x] nenastavovala som pre existujúci uploader súborov do ktorého blob storage sa majú nahrávať, teraz keď máme 2 storage asi to bude treba jasne špecifikovať aj pre neho
  • [x] typescript ma varuje pred tým, že si posúvam dve funkcie do use client komponenty (UploadImage) ako props, naoko to funguje ako má ale neviem posúdiť ako veľmi nebezpečné to je... Posúvam si setter stavu avatarImage a funkciu na update foto v databáze (onAvatarChange)
  • [x] neviem úspešne docieliť aby sa nová fotka zobrazovala všade kde používame avatara, napr navbar alebo v detaile projektu... jeden z problémov je, že bude treba nastaviť presmerovanie z https://mogrfyhmal8klgqy.public.blob.vercel-storage.com/:file na assets.cesko.digital/:file plus sa pohrať s typmi (nie všade stačí nahradiť rovno avatarUrl za profilePictureUrl..)

laurabaluchova avatar Sep 25 '24 13:09 laurabaluchova

Update:

  • po novom funguje nová fotka v navbare (riešené cez session image update), tiež v detaile profilu a vo výpise ľudí

  • tiež je už blob storage URL povolené, ale zatiaľ len cez next config, nie elegantne cez redirect, do cd-tools som nešahala, takže:

  • [ ] pred merge by bolo potrebné nastaviť redirect a upraviť next config

Čo som nevedela urobiť:

  • [ ] neviem nastaviť novú foto vo výpise ľudí v projekte, myslím, že bude potrebná zmena v databáze nech sa má ako dekódovať profilePictureUrl.. Začala som to ale všade je zatiaľ default avatar, aj u mňa, pričom mám profilovku nastavenú už cez profilePictureUrl pole
  • [ ] neviem prestaviť foto ownera metriky, tiež tuším nutnosť zmeny v databáze?
  • [ ] Niektoré stránky detailov projektu mi nechce otvoriť, napr. Festival kreativní byrokracie alebo Digitální inkluze, neviem, či som tam niečo rozbila teraz...

laurabaluchova avatar Sep 29 '24 20:09 laurabaluchova

Ten původní picker je super, ale ty následující změny už jsou trochu brute-force a já se v nich ztrácím :), takže tenhle PR zavřu a bokem udělám nový, do kterého si cíleně vyzobu jen něco. Pardon za komplikace!

zoul avatar Oct 28 '24 09:10 zoul