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.

Sign up for GitHub

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

Jump to bottom

Bug data #155

Merged
merged 11 commits into from
Feb 15, 2024
Merged

Bug data #155

merged 11 commits into from
Feb 15, 2024

Conversation

angeloneves19
Copy link
Contributor

148 Bug: Data

🆙 CHANGELOG

  • Antes havia um problema de dias da semana e data -1, então resolvemos da seguinte maneira: Melhoramos a função format-data dentro de helpers e refatoramos, e usamos ela na descrição de eventos, oficinas e terapias, reutilizando-a como uma Constant global por ser o mesmo código.

⚠️ Me certifico que:

  • Não deixei nenhum novo warning, erro ou console.log nas minhas modificações
  • Fiz deploy para ambiente de teste certificando que o build não quebrou
  • Solicitei code review para 2 pessoas
  • Solicitei QA para 2 pessoas
  • Obtive aprovação de QA e posso fazer merge

⚠️ Como testar:

  • Acessar a branch bug_data.
  • Rodar o projeto no CMS e Front-end.
  • Verificar se as datas adicionadas no Strapi, condizem com as datas das descrições de eventos, oficinas e terapias.
  • Fazer deploy em ambiente de teste
  • Abrir URL da aplicação de teste
  • Aplicação não deve conter nenhum erro, warning ou console.log
  • Alteração proposta no card foi implementada

@angeloneves19
Refactor(src/components/events-carrosel/index.js): adicionamos um tim…
b3c5ef5 
…er na funcao new Date -@angeloneves19 -@thayanneregoo
@angeloneves19
feat(src/helpers/format-data/index.js): adicionamos um timer no final…
fb51d98 
… do mapeamento adicionando as 3horas que faltavam, que o new Date nao tinha
@angeloneves19
feat(src/pages/therapies-description/index.js): adicionamos o timer d…
f3bab4b 
…entro do new date para adicinar as 3horas que o new date nao tem! -@angeloneves19 -@thayanneregoo -@Ester-Arruda
@angeloneves19
feat(src/pages/workshop-description/index.js): adicionamos o timer de…
6cfaa06 
…ntro do new date para adicinar as 3horas que o new date nao tem! -@angeloneves19 -@thayanneregoo -@Ester-Arruda
@angeloneves19
feat(src/compontents/events-carrosel/index.js/helpers/format-data/ind…
ab772d0 
…ex.js/pages/events-description/index.js): Refatora funcao formatDate exibe hora em evento -@angeloneves19 -@thayanneregoo -@Ester-Arruda
@angeloneves19
Refactor(src/pages/therapies-description/index.js/workshop-descriptio…
bbf3b36 
…n/index.js): ajustando bug da data na terapias e oficinas -@angeloneves19 -@thayanneregoo
@angeloneves19 angeloneves19 merged commit 8b9a386 into main Feb 15, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yrachid yrachid yrachid left review comments

@daniellemadrid daniellemadrid daniellemadrid approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@angeloneves19 @yrachid @daniellemadrid @fernandesmelo