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

#192 dashboard-slides #35

Merged
merged 20 commits into from
Nov 23, 2019
Merged

#192 dashboard-slides #35

merged 20 commits into from
Nov 23, 2019

Conversation

KiSobral
Copy link
Member

Descrição

Os slides do dashboard foram reorganizados para que o segundo elemento do slider sejam as colheitas do usuário. Também foi reestilizado o componente de "HarvestDigest" criado para visualização rápida de colheitas.

Resolve (Issues)

192

Tarefas gerais realizadas

  • Segundo slide colheitas da semana (JÁ ESTÁ IMPLEMENTADO, APENAS COLOCAR NA ORDEM);
  • Adicionar link em cada cartão de colheita para redirecionar para a página da colheita;
  • Adicionar o slide de minhas colheitas;
  • Resgatar do backend todas as colheitas do proprietário;

@KiSobral KiSobral changed the title [WIP] #192 dashboard-slides #192 dashboard-slides Nov 22, 2019
Copy link
Collaborator

@shayanealcantara shayanealcantara left a comment

Choose a reason for hiding this comment

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

@KiSobral, ta tudo muito bom 💜 , eu só senti falta de uma mensagem no slide de "Suas colheitas" quando ainda não foi cadastrada nenhuma colheita. Acho que uma mensagem de que "Você não cadastrou nenhuma colheita até o momento", desse tipo seria ótimo pra padronizar como as outras que tem uma mensagem de nenhuma propriedade encontrada, por exemplo.

@mardntas mardntas changed the title #192 dashboard-slides [WIP]#192 dashboard-slides Nov 22, 2019
@KiSobral
Copy link
Member Author

@KiSobral, ta tudo muito bom purple_heart , eu só senti falta de uma mensagem no slide de "Suas colheitas" quando ainda não foi cadastrada nenhuma colheita. Acho que uma mensagem de que "Você não cadastrou nenhuma colheita até o momento", desse tipo seria ótimo pra padronizar como as outras que tem uma mensagem de nenhuma propriedade encontrada, por exemplo.

A gente já vai ajeitar essa parte do issue e depois disso esse baby de PR já vai poder subir :)

Copy link
Collaborator

@shayanealcantara shayanealcantara left a comment

Choose a reason for hiding this comment

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

Ficou top, parabéns!!!! 🤸‍♀️
Sobre os checks que não passaram: estamos vendo o que rolou, mas até então não parece ser o código em si (espero) hehe

@mardntas
Copy link
Collaborator

@KiSobral, ta tudo muito bom , eu só senti falta de uma mensagem no slide de "Suas colheitas" quando ainda não foi cadastrada nenhuma colheita. Acho que uma mensagem de que "Você não cadastrou nenhuma colheita até o momento", desse tipo seria ótimo pra padronizar como as outras que tem uma mensagem de nenhuma propriedade encontrada, por exemplo.

Feito!

@mardntas mardntas changed the title [WIP]#192 dashboard-slides #192 dashboard-slides Nov 23, 2019
@shayanealcantara
Copy link
Collaborator

shayanealcantara commented Nov 23, 2019

Ficou top, parabéns!!!! woman_cartwheeling
Sobre os checks que não passaram: estamos vendo o que rolou, mas até então não parece ser o código em si (espero) hehe

Achamos a causa do problema: um bug recentemente gerado em uma nova release, na dependência vuetify. O bug está resolvido na branch feature/179. Para quem for avaliar este PR, pode desconsiderar qualquer log referente ao erro 137 nos checks da integração contínua.

@mardntas mardntas changed the title #192 dashboard-slides [WIP]#192 dashboard-slides Nov 23, 2019
@KiSobral KiSobral changed the title [WIP]#192 dashboard-slides #192 dashboard-slides Nov 23, 2019
Copy link
Collaborator

@LeoSilvaGomes-zz LeoSilvaGomes-zz left a comment

Choose a reason for hiding this comment

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

Lindissimo ❤️

@fabiolamfleury fabiolamfleury merged commit 6a1471c into develop Nov 23, 2019
@fabiolamfleury fabiolamfleury deleted the feature/192-dashboard-slides branch November 23, 2019 18:40
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.

6 participants