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 smazání profilové fotky #1120

Open
zoul opened this issue Oct 31, 2024 · 0 comments
Open

Přidat možnost smazání profilové fotky #1120

zoul opened this issue Oct 31, 2024 · 0 comments
Labels
contributions-welcome Věci, co jdou udělat i bez velkého organizačního a produktového kontextu topic:user-profile Uživatelský profil

Comments

@zoul
Copy link
Member

zoul commented Oct 31, 2024

V rámci #1119 jsme přidali možnost nastavit profilovou fotku, ale chybí možnost tu fotku smazat. Z hlediska UI je to triviální, ale je potřeba domyslet aktualizaci na serveru. Aktuálně má ta property typ string | undefined, což dává smysl, ale když voláme PATCH /account/me, tak nastavení profilovky na undefined (žádná profilovka) vlastně znamená, že tu property nechceme aktualizovat. Možná by ten typ měl být spíš string | null? Měly by pak ale být všechny volitelné properties taky typu X | null namísto X | undefined?

@zoul zoul added the topic:user-profile Uživatelský profil label Oct 31, 2024
@zoul zoul added the contributions-welcome Věci, co jdou udělat i bez velkého organizačního a produktového kontextu label Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributions-welcome Věci, co jdou udělat i bez velkého organizačního a produktového kontextu topic:user-profile Uživatelský profil
Projects
None yet
Development

No branches or pull requests

1 participant