Bug data #154

angeloneves19 · 2024-02-06T13:07:45Z

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

…ront into 100A/Responsividade-Breakpoints-Navbar

…ront into production

@angeloneves19

…er na funcao new Date -@angeloneves19 -@thayanneregoo

… do mapeamento adicionando as 3horas que faltavam, que o new Date nao tinha

@angeloneves19

…entro do new date para adicinar as 3horas que o new date nao tem! -@angeloneves19 -@thayanneregoo -@Ester-Arruda

@angeloneves19

…ntro do new date para adicinar as 3horas que o new date nao tem! -@angeloneves19 -@thayanneregoo -@Ester-Arruda

yrachid · 2024-02-07T13:13:26Z

src/components/events-carrosel/index.js

@@ -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>


Suggested change

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

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

yrachid · 2024-02-07T13:15:40Z

src/components/events-carrosel/index.js

@@ -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`),


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?

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.

yrachid · 2024-02-07T13:17:33Z

src/helpers/format-data/index.js

-      : `${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`))} 


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.

yrachid · 2024-02-07T13:25:43Z

src/components/events-carrosel/index.js

@@ -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`),


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.

@angeloneves19

…ex.js/pages/events-description/index.js): Refatora funcao formatDate exibe hora em evento -@angeloneves19 -@thayanneregoo -@Ester-Arruda

@angeloneves19

…n/index.js): ajustando bug da data na terapias e oficinas -@angeloneves19 -@thayanneregoo

angeloneves19 added 11 commits December 11, 2023 14:59

feat():puxando alteracoes da mainn -@angeloneves19 -@thayaneregoo

f6faf62

Merge branch 'main' into production

133b46a

Merge branch 'main' of https://github.com/aceleradora-TW/nossa-casa-f…

7a1bc56

…ront into 100A/Responsividade-Breakpoints-Navbar

arrumando conflitos -@angeloneves19 -@thayaneregoo

d68e22d

Merge branch 'main' of https://github.com/aceleradora-TW/nossa-casa-f…

1930424

…ront into production

Updated Yarn lockfile

b5b01a5

Merge branch 'main' of https://github.com/aceleradora-TW/nossa-casa-f…

d810c4e

…ront into production

Refactor(src/components/events-carrosel/index.js): adicionamos um tim…

b3c5ef5

…er na funcao new Date -@angeloneves19 -@thayanneregoo

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

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

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

daniellemadrid approved these changes Feb 6, 2024

View reviewed changes

yrachid reviewed Feb 7, 2024

View reviewed changes

yrachid requested changes Feb 7, 2024

View reviewed changes

angeloneves19 added 2 commits February 9, 2024 15:56

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

Refactor(src/pages/therapies-description/index.js/workshop-descriptio…

bbf3b36

…n/index.js): ajustando bug da data na terapias e oficinas -@angeloneves19 -@thayanneregoo

angeloneves19 closed this Feb 12, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Bug data #154

Bug data #154

angeloneves19 commented Feb 6, 2024

yrachid Feb 7, 2024

yrachid Feb 7, 2024

yrachid Feb 7, 2024 •

edited

Loading

yrachid Feb 7, 2024

yrachid Feb 7, 2024 •

edited

Loading

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

Bug data #154

Bug data #154

Conversation

angeloneves19 commented Feb 6, 2024

148 Bug: Data

🆙 CHANGELOG

⚠️ Me certifico que:

⚠️ Como testar:

yrachid Feb 7, 2024

Choose a reason for hiding this comment

yrachid Feb 7, 2024

Choose a reason for hiding this comment

yrachid Feb 7, 2024 • edited Loading

Choose a reason for hiding this comment

yrachid Feb 7, 2024

Choose a reason for hiding this comment

yrachid Feb 7, 2024 • edited Loading

Choose a reason for hiding this comment

yrachid Feb 7, 2024 •

edited

Loading

yrachid Feb 7, 2024 •

edited

Loading