Feature/associate categories #805
Conversation
✅ Deploy Preview for cambiatus-elm-book ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy done!
To edit notification comments on pull requests, go to your Netlify site settings. |
|
We're still having some issues that have to be addressed on the backend, but other than that, everything should be working already |
|
Bro, do you think we can hide the category editing option if the community has no categories? |
|
Done! ⚡ |
|
OI @NeoVier , por favor, você me avisa quando for para testar? |
|
Já pode testar @juramos-2020 😁 |
|
OI @NeoVier seguem alguns feedbacks:
Quando clico, em uma segunda vez, em "Editar categorias dessa oferta" da mesma oferta, já aparece os indicadores de steps conforme design
Acho que esses indicadores deveriam aparecer desde a primeira vez que clicamos em "Editar categorias dessa oferta". OBs: Isso acontece quando eu estou logada como Admin e tento editar uma oferta de outra pessoa. Quando clico em "Editar categorias dessa oferta" de uma oferta minha (admin) os steps aparecem corretamente conforme o design. |
|
Segue os demais itens: botão para compartilhar uma oferta ✅ Se você for o administrador da comunidade, também poderá clicar nela para editar as categorias de uma oferta (se a comunidade não tiver categorias, o administrador deverá ver apenas o botão de compartilhamento) ✅ Se não for admin da comunidade, a oferta deve apresentar apenas o botão de compártilhar ✅ Associar um produto a uma categoria ✅ Ainda não temos como filtrá-los, então você precisa voltar ao formulário e ver se as categorias continuam marcadas ✅ Teste-o em uma comunidade que não tenha categorias (a etapa de categorias não deve aparecer) e verifique se ainda funciona normalmente ✅ Faça login como administrador da comunidade e verifique se você pode editar as categorias de outras pessoas ✅ Como administrador de uma comunidade que não tem categorias, certifique-se de que a opção de editar categorias das ofertas das pessoas não apareça ✅ Ainda vou analisar o mobile para ver se teve alguma quebra de página... |
|
Uma dúvida @NeoVier @lucca65 esse cenário de criação de uma oferta com categorias vai entrar na entrega do item 2 ou do item 3 ? 👇
|
Discutimos isso aqui no figma, e decidimos não mostrar os steps para o admin, já que ele não pode editar os outros campos da oferta. Os steps apareceram na segunda vez por conta de um bug que fazia com que a oferta se tornasse sua ao editar as categorias como admin. Já temos a correção, mas o backend de staging estava em outra versão. Mudei a versão do backend de staging, então não deve mais acontecer.
Não entendi @juramos-2020. Esse link manda no cenário de recomendação/boosting de ofertas. O item 1 já foi feito (e já fizemos deploy em produção). Nesse PR já adicionamos o item 2 e a possibilidade de criar ofertas com categorias. O item 3 vem em #806
Vou verificar 🏃♂️ |
Esse link aqui: |
Nesse PR já é para testar a possibilidade de criar ofertas com categoria? |
|
Sim 😁 |
O meu comentário no figma era sobre o botão de "continue" Mas tranquilo, não vejo problema de não termos esse elemento visual visto que o admin não vai precisa passar pelos outos steps. Mas é bom saber para atualizar o Figma 😊 |
|
@NeoVier então no caso do admin editar uma oferta de outra pessoa, os steps não aparecem, correto? |
|
Isso mesmo, os steps aparecem para o dono do produto, sendo admin ou não |
|
Valeu @NeoVier então, recapitulando, seguem todos os cenarios que testei na versão desktop. Todos funcionam normalmente com excessão de não conseguir criar uma categoria (
Vou fazer o teste na versão na versão Mobile 😎 |
|
@NeoVier segue feedback na versão mobile:
|
@juramos-2020 não foi resolvida essa issue ainda. Como é algo que vai ter uma complexidade razoável e não tem um impacto tão grande deixamos um pouco de lado. Dito isso. Grande parte das imagens que estão nas comunidades devem funcionar normalmente. Não tenho certeza o que aconteceu com o preview que está no print. Fui testar e tanto no Slack quanto no Telegram há uma imagem, mas não a mesma que está listada no produto. Talvez seja algo a ver com o netlify. Alguma ideia @NeoVier?
|
|
Acho que é por conta do Netlify mesmo. O link aponta para os servidores deles, e não os nossos, então não conseguimos interceptar a request pra servir o HTML gerado pelo backend que mostraria o preview bonitinho |
|
Fiz o deploy do frontend no staging, então podem compartilhar com o link
|
Não consegui reproduzir... Qual é a mensagem de erro exatamente, e quando ela aparece? |
Tranquilo @MatheusBuss
Pra mim ainda aparece dessa forma 👇. Mas assim, o compartilhamento está funcinando pois está compartilhando. Só estava na dúvida sobre a aparencia desse compartilhamento. Sabia que o @MatheusBuss estava trabalhando nisso só não tinha certeza se já tinha sido concluído ou não. Então fica apenas o registro de como tá o compartilhamento para documentar mesmo. |
|
É que o link tá errado Ju 😅. Pro preview do link funcionar tem que ser pelo link do frontend de staging (e não pelo gerado aqui no Github). No caso desse produto é https://nomad.staging.cambiatus.io/shop/74
|
@NeoVier Tentei reproduzir agora para filmar mas não aconteceu, acho que de alguma forma foi resolvido, que bom rsrs 😎 |
|
Bom, o único problema então é na hora de criar categoria... Como isso não está relacionado a esse PR, vou fazer o merge desse e vemos isso em outro PR. Já sinalizei o problema aqui no Slack: https://cambiatus.slack.com/archives/CA83HJAAD/p1660648202151329 |
Ahhhh bom, assim sim hahaha 😂 Arrasaram meninos ficou show!!!!
|
Combinado! |
|
Animal pessoal, vamos ajustar essas arestas aí, mas to contente que nessa semana sai nosso querido update final de categorias!! |
What issue does this PR close
Closes #770
Changes Proposed ( a list of new changes introduced by this PR)
How to test ( a list of instructions on how to test this PR)