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

Bug data #154

Closed
wants to merge 13 commits into from
Closed

Bug data #154

wants to merge 13 commits into from

Conversation

angeloneves19
Copy link
Contributor

148 Bug: Data

🆙 CHANGELOG

  • Adicionamos a string GMT dentro do New Date( ) no carrossel de eventos, na descrição de eventos, oficinas e terapias.

⚠️ 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 dos 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

@@ -65,7 +65,7 @@ const Events = () => {
<img className="img" src={events.image_url} />
</div>
<div>
<p className="date">{events.date.toLocaleDateString('pt-BR', { Timezone: 'UTF' })}</p>
<p className="date">{events.date.toLocaleDateString('pt-BR', { Timezone: 'UTC' })}</p>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<p className="date">{events.date.toLocaleDateString('pt-BR', { Timezone: 'UTC' })}</p>
<p className="date">{events.date.toLocaleDateString('pt-BR', { timeZone: 'UTC' })}</p>

@@ -19,21 +18,22 @@ const Events = () => {
return {
id: data.id,
name: data.attributes.nome,
date: new Date(data.attributes.data_inicio),
date: new Date(`${data.attributes.data_inicio}T00:00:00-0300`),
Copy link
Member

Choose a reason for hiding this comment

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

Porque estamos adicionando horário e offset de timezone? Qual é o bug que ocorre quando esta informação não está presente?

O que acontecerá quando o Brasil entrar em horário de verão e o offset mudar para -0200?

Copy link
Member

@yrachid yrachid Feb 7, 2024

Choose a reason for hiding this comment

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

Preciso entender melhor o problema que está ocorrendo, mas tenho a impressão de que concatenar um offset de timezone não será a solução.

Acho que deveríamos continuar lidando com datas em UTC e só mudarmos o offset na hora de exibir o valor da data. Já fazemos isso no código desse componente, só estamos fazendo errado hehehe:

Mais para baixo, fazemos:

events.date.toLocaleDateString('pt-BR', { Timezone: 'UTC' }

Quando poderíamos fazer:

events.date.toLocaleDateString('pt-BR', { timeZone: 'America/Sao_Paulo' }

Isso vai ajustar a data para o horário de Brasília e já vai dar conta de converter corretamente horário de verão. Já havia falado sobre isso em uma PR bem antiga.

: `${formatDate(new Date(workshop.attributes?.data_inicio))}
> ${formatDate(new Date(workshop.attributes?.data_fim))}`
? formatDate(new Date(`${workshop.attributes?.data_inicio}T00:00:00-0300`))
: `${formatDate(new Date(`${workshop.attributes?.data_inicio}T00:00:00-0300`))}
Copy link
Member

Choose a reason for hiding this comment

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

Essa correção está se repetindo em tudo que é canto. Não seria melhor deixar a função formatDate fazer isso internamente? Continuamos passando a string de data para ela e ela concatena a informação faltante, assim não precisamos repetir a mesma coisa toda vez que a função for chamada.

@@ -19,21 +18,22 @@ const Events = () => {
return {
id: data.id,
name: data.attributes.nome,
date: new Date(data.attributes.data_inicio),
date: new Date(`${data.attributes.data_inicio}T00:00:00-0300`),
Copy link
Member

@yrachid yrachid Feb 7, 2024

Choose a reason for hiding this comment

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

Preciso entender melhor o problema que está ocorrendo, mas tenho a impressão de que concatenar um offset de timezone não será a solução.

Acho que deveríamos continuar lidando com datas em UTC e só mudarmos o offset na hora de exibir o valor da data. Já fazemos isso no código desse componente, só estamos fazendo errado hehehe:

Mais para baixo, fazemos:

events.date.toLocaleDateString('pt-BR', { Timezone: 'UTC' }

Quando poderíamos fazer:

events.date.toLocaleDateString('pt-BR', { timeZone: 'America/Sao_Paulo' }

Isso vai ajustar a data para o horário de Brasília e já vai dar conta de converter corretamente horário de verão. Já havia falado sobre isso em uma PR bem antiga.

…ex.js/pages/events-description/index.js): Refatora funcao formatDate exibe hora em evento -@angeloneves19 -@thayanneregoo -@Ester-Arruda
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.

3 participants