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

Feature/product categories list #9

Open
wants to merge 45 commits into
base: develop
Choose a base branch
from

Conversation

bydlowski
Copy link

  • Criação do mockdata;
  • Criação do store;
  • Mudanças no Header;
  • Criação do componente de Menu e enuCategory;
  • Mudanças para usar typescript;

@bydlowski
Copy link
Author

PS: se der, dá uma checada geral no que eu foz com typescript. Faz tempo que não usava.
PS2: não fiz mta mudança de layout. Qualquer coisa me falem q eu faço alguma alteração.

@bydlowski bydlowski linked an issue Oct 19, 2020 that may be closed by this pull request
7 tasks
@@ -9,7 +9,10 @@
# install dependencies
$ yarn install

# serve with hot reload at localhost:3000
# serve mockdata with hot reload at localhost:3000
$ yarn mock

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@@ -0,0 +1,20 @@
<template>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tava pensando que aqui poderíamos usar o componente Navigation-Drawer, no lugar de v-menu , ele se parece mais com o tipo de menu lateral que normalmente é utilizado.
https://next.vuetifyjs.com/en/components/navigation-drawers/

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Podemos mudar sim! Eu só segui o que já estava meio que iniciado. Vejam o que acham melhor.
Talvez podemos criar um novo card pra fazer essa alteração pq esse já lida com a estrutura pra pegar as categorias na api, criar subcategorias, etc.

}

created() {
this.$accessor.app.getCategories()

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

O Fetch das categorias acho que poderíamos fazer via Server Side Rendering, dai quando o cliente entrasse no site elas seriam buscadas no backend e o cliente não veria o loading..., depois, por motivos de SEO vamos precisar fazer isso pra lista de produtos também. Da uma olhada nisso pra ver se ajuda entender: https://nuxtjs.org/guides/features/data-fetching/

ps.: SSR pode ser feito em outro PR. ;)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isso, tava no meu plano sim converter pra SSR. Vou criar um card lá pra implementar essa mudança.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

As an user, I want to see a list of product categories
3 participants