Feature/shop categories #760
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. |
|
@lucca65 I have a few implementation questions, can you help me out?
|
Bro, I'm somewhat out of context, but I'll adjust the backend to cascade the deletion in this case. This way you only worry about sending a single mutation that will automatically delete all children
My vote is for markdown
Well I've added those as optional, so we don't really have to fill it. It would be nice to have this from the get go, but we would go astray, without the design team, might be a big step. I can visualize how it would look like on the end, with very similar results to a table with inputs and modals to add information, but I'll leave this one to you, please organize and choose the features the way you see fit
To be honest I haven't even thought about it... I thought it would simply URL encode characters it won't knew how to properly render. the internet standards are not very inclusive I guess... When rendering the slugs we have to keep in mind that this is not something for us, but for the indexing bots. We only need the ID. I think its a fair tradeoff to loose some visibility to google spiders 👍 |
|
oh, one more thing... I'm adding more tests to the categories/subcategories relationships and it might change the api a little bit... the way I'm creating self references isn't the most used one for relational databases so it might take some tweaking! I'll ping you when its done |
@juramos-2020 esse é o espaçamento que usamos em todos os modais... Quer aumentar o espaçamento só nesse modal ou em todos? @lucca65 o erro para criar categoria é por causa do position, que é obrigatória. Devemos mandar algo como 0, ou o vai ter alguma alteração no backend? |
|
pode mandar como 0. ao deletar está dando um erro, vou arrumar e criar um caso de teste também |
Henrique, se for fácil mudar em todos (24px apenas para desktop) pode mudar. O nosso DS ta meio desajustado no momento mas vamos trabalhar para padronizar esses componentes🏃♂️ . Se for muito trabalhoso mudar em todos podemos ajustar só esse e atualizamos os demais em uma próxima oportunidade. |
|
Não é trabalhoso, mas eu diria que é melhor deixarmos isso para outro PR, onde podemos testar melhor os outros modais, pra garantir que nada quebre ou fique estranho. Por enquanto, ajustei só nesse modal |
|
@NeoVier @juramos-2020 arrumei a deleção, agora ao apagar uma categoria pai, também seus filhos e netos são apagados automaticamente. Já fiz o deploy em staging. |
sucesso 💪 |
Isso quer dizer que devo mudar o link que estou fazendo os testes? Por enquanto estou usando esse do deploy |
|
Pode continuar usando esse link mesmo Ju |
|
Oi @lucca65 @NeoVier achei essa referência do Asana bem legal sobre a questão de ordenar para cima e para baixo. Estou compartilhando com o intuito de ajudar nas pesquisas e estudos que vocês estão fazendo |
|
O upload está dando erro através do celular. Vou tentar fazer o upload mais tarde através do notebook |
|
Consegui gerar um link do Google fotos 😅 https://photos.app.goo.gl/e6PZsiRWW6KuaHTm7 |
|
@juramos-2020 deixei esses comentários na issue que vamos tratar o ordenamento: #780 (comment) |
|
Olá @NeoVier
Tá demais 🐱👤🚀
Criei 3 categorias e subcategorias 1. Alimentos (obs.: temos como colocar apenas as categorias com o texto em negrito? para termos uma diferenciação entre as categorias e subcategorias?) No desktop: Tanto com o "arrastar" do mouse como usando os iconês de "mover para cima" e "mover para baixo" eu só consigo mover uma subcategoria se for para dentro de outra subcategoria ou para dentro de outra categoria. Não consigo fazer com que o "orgânicos", por exemplo, fique em cima de "massas". Estou na dúvida se esses são os comportamentos esperados pois vocês comentaram "Removi a parte de ordenação, deixando só a funcionalidade de trocar de pais". Eu tinha entendido algo sobre isso mas acho que entendi errado. Então, por favor, você pode me explicar melhor qual seria o comportamento esperado com isso? No mobile: o comportamento de pressionar com os dedos e mover não está funcionando muito bem, tem momentos que funciona outros não. Vou tentar com outro celular 😉 Tiveram alguns momentos em que eu cliquei naquele menu, que estamos usando com os "3 pontinhos", e por algumas vezes ele abrir em diferentes espaços na tela. Então temos algumas situações para observar/corrigir: 1 - Quando as opções do menu aparecem "cortadas",
|
Por enquanto, esse é o comportamento esperado. Vamos adicionar a ordenação em #780.
Ordenação é a habilidade de escolher a ordem em que as categorias vão aparecer (por exemplo, massas antes de orgânicos, ou orgânicos antes de massas). Até resolvermos #780, não teremos essa habilidade. A única coisa que podemos fazer é trocar o pai de uma categoria (por exemplo, fazer com que massas seja uma subcategoria de "Produtos Pets"). As categorias serão ordenadas por ordem alfabética dentro de cada contexto (ou seja, as categorias "Alimentos", "Produtos Pets" e "Mover" serão organizadas por ordem alfabética. Dentro de "Alimentos", as categorias massas, orgânicos e vitaminas serão organizadas por ordem alfabética, e assim por diante)
Não vejo motivos para não deixar transformar sub-categorias em categorias e vice-versa (caso tenha algum motivo em mente, sinta-se à vontade para explicar 😁). Dito isso, não existe uma diferenciação entre categoria e sub-categoria (é tudo categoria, a única diferença é que algumas tem categorias pais, outras não).
Vou alterar
Acredito que isso seja por causa do comportamento de abrir o menu com o botão direito do mouse. No celular, isso acontece ao manter pressionado. Vou arrumar! Também vou resolver a questão de ele ficar cortado, fazendo com que ele apareça para o outro lado (ao invés de aparecer à direita do mouse, aparecer à esquerda)
Para economizar espaço, o ícone de edição entra no menu dos três pontinhos em telas menores, como mencionado nos comentários anteriores. |
|
@juramos-2020 ajustes feitos ✅ |
Sucesso Henrique obrigada pela explicação 😍
Sobre isso não é um problema não. Só estava reportando pois estava na dúvida se vcs estavam esperando esse comportamento.
Sucesso então |
|
Valeu @lucca65 e @juramos-2020 pela paciência, esse PR foi uma bela duma luta 😁 |
|
Você Arrasou demais Henrique 😻😻😻 🎉💯💚🤹🧞 Queria colocar um Gif animado mas não tô conseguindo 😂😂😂😂 |
|
Ufa pessoal! parabéns! foi um baita de um PR memorável! Vamos comemorar pq é merecido! |
|
Imagina Henrique, nada de paciência senão eu posso dizer o mesmo hahahahaha Você fez um lindo trabalho 😍 |
What issue does this PR close
Closes #754
Changes Proposed ( a list of new changes introduced by this PR)
/community/settings/shop/categories)How to test ( a list of instructions on how to test this PR)
...button on each category)...button)