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

Přidat možnost nastavení profilovky #1055

Closed
zoul opened this issue Jul 10, 2024 · 5 comments · Fixed by #1119
Closed

Přidat možnost nastavení profilovky #1055

zoul opened this issue Jul 10, 2024 · 5 comments · Fixed by #1119
Assignees
Labels
topic:user-profile Uživatelský profil

Comments

@zoul
Copy link
Member

zoul commented Jul 10, 2024

Doposud jsme podporovali pouze nastavení profilovky přes Slack, výhledově budeme chtít přidat možnost nastavení profilovky přímo – zejména protože do Slacku už nám lidi nepropadávají. Zvládnem uchovat ten Slack jako fallback?

PS. A případně bychom zároveň s tímhle mohli spláchnout nastavení jména?

@zoul zoul added the topic:user-profile Uživatelský profil label Jul 10, 2024
@anezkamll
Copy link
Contributor

@zoul nastavení jména už v profilu je, můžeme dodělat tohle? Opět mi na to dnes přišel požadavek 🙏

@zoul
Copy link
Member Author

zoul commented Sep 16, 2024

Určitě to můžem dodělat, je to jen otázka prioritizace. Můžeš na to myslet, prosím, a dostaneme se k tomu po prvním odbavení hashtagů? Rád bych ty hashtagy vzal prioritně, je v tom hodně kódu a změn v databázi, které se dlouho táhnou a rád bych je už měl za sebou. Případně by si tohle mohl vzít na starost Kuba, jak bude mít čas mimo #1094.

@anezkamll
Copy link
Contributor

Jasně, určitě. Mně to vypadlo ze zřetele, protože jsme sjednotili pole z formuláře a v editaci, nějak mi uteklo, že fotku to nezahrnuje. 😇

@anezkamll
Copy link
Contributor

👀 @laurabaluchova zde ticket na fotky

@zoul
Copy link
Member Author

zoul commented Sep 18, 2024

Poznámky:

  • Stávající fotky se načítají z pole slackAvatarUrl v tabulce User Profiles, to pole samotné se plní přes vazbu slackUser z tabulky Slack Users (linked records). Je potřeba namyslet, jak s touhle existující fotkou pracovat. Můžeme ju použít jako výchozí hodnotu, tedy že by to pro tyhle lidi vypadalo, jako kdyby si už tu profilovku nastavili někdy dřív ručně?
  • Nové obrázky chceme načítat do Vercel Blobu. Nahrávat nemůžeme jednoduše z klienta, protože by leaknul API klíč, takže je potřeba dopsat API endpoint, který oveří přihlášení uživatele, zkontroluje data a pokud bude všechno sedět, nahraje soubor do blob storage a vrátí URL k obrázku, které pak můžem uložit do databáze. (Anebo ten endpoint rovnou asi uloží URL obrázku do databáze k dotyčnému uživateli? Tedy něco jako POST /account/picture?)
  • Jednu stávající blob storage máme, ale pro tyhle data pocházející od běžných uživatelů bych založil novou, ať to nemícháme. Pro inspiraci lze omrknout tenhle PR pro uploadovací nástroj: Nástroj pro snadný upload obrázků a dalších příloh #1033. (My jsme pak přidali upload přímo z klienta, viz Přidat do uploadovacího nástroje upload přímo z klienta #1035, ale ten mně tady nepřijde nutný; ten upload ze serveru je koncepčně o něco jednodušší.)
  • Bude dobré kontrolovat velikost a typ souborů, ať nám tam lidi nenahrávají zbytečně obří věci. Upload ze serveru je stejně omezený zhruba na 4,5 MB.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic:user-profile Uživatelský profil
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants