Feat: adiciona uma descrição ao usuário #1470
Conversation
|
@ValbertMartins is attempting to deploy a commit to the TabNews Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Fala @ValbertMartins, obrigado pelo PR! 💪
Sei que você ainda vai fazer modificações, como adicionar testes, mas já fiz algumas perguntas no código 👍
|
Nova sugestão de mudança para a ui do
o que acham?
|
There was a problem hiding this comment.
Show @ValbertMartins! 💪
Fiz comentários/sugestões no código. Por favor, veja se fazem sentido. 🤝
pages/perfil/index.public.js
Outdated
| clearErrors(); | ||
| setDescription(value); | ||
| }} | ||
| value={user?.description || ''} |
There was a problem hiding this comment.
| value={user?.description || ''} | |
| value={description} |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@aprendendofelipe e demais, ainda não cheguei a um número mais concreto de limite para a descrição, no momento está A ideia com esse número é fazer com que os usuários sejam mais objetivos na descrição, acredito que esse número ainda pode aumentar, porém aumentar muito mais que isso talvez não seja tão bom, o que acham? |
Acho que 5.000 pode até ser muito alto para testar a funcionalidade, pois pode ser complicado se precisarmos diminuir o número no futuro. Como exemplo, é quase o dobro do permitido para a descrição no LinkedIn, que é 2.600 (2.000 no caso de descrição dos cargos). Por outro lado, nós vamos aceitar markdown, então nem sempre todos os caracteres serão visíveis. Vamos testar 5.000 mesmo em homologação? Mais tarde vou rodar a migration. |
|
Já rodei a migration @ValbertMartins! 🚀 https://tabnews-git-fork-valbertmartins-feat-user-description-tabnews.vercel.app/FelipeBarso
O PR ainda está como draft. Você ainda vai fazer alguma modificação ou já está pronto? |
|
@aprendendofelipe sensacional, acho que chegamos numa versão pronta pra ser testada! ✅ |
|
Que trabalho sensacional em comunidade 😍 |
|
@ValbertMartins, você foi citado no post comemorativo: https://www.tabnews.com.br/FelipeBarso/tabnews-6-milhoes-de-visualizacoes-e-muita-novidade 🎉 |
Esse PR, adiciona um draft inicial da feature de adicionar descrição ao usuários.
Alguns pontos iniciais a se destacar
• Com a sugestão do @filipedeschamps de ser possível usar markdown mudei um pouco a ui para um box, porém isso ainda é um esboço inicial da ui
•
irei fixar os testes, os testes que estão quebrando são aqueles que retornam estritamente um usuário💪✅•
A quantidade inicial está como 160 caracteres na migration, mas acredito que agora esse número é pouco.✅• preview perfil
• preview caixa descrição
