73/feat eventos mostrar atividades #112
Conversation
…tion\Caroulsel.jsx): exibindo informações no carrosel, criando componente de carrosel, iniciando renderização condicional da seção de atividades (subeventos e oficinas) - @claudionsc - @thayanerego
…zanção condicional das atividades (subeventos e oficinas) - @claudionsc - @thayanerego
…raçaõ de dados de oficinas (oficinas) - @claudionsc - @thayanerego
…carrossel - @claudionsc - @thayanerego
…das suboficinas e subeventos da página de descrição de eventos, alterando rota oficina para workshop - @claudionsc - @thayanerego
…tion\Caroulsel.jsx): exibindo data conforme verificação de duração de evento, exibindo horaraio de inicio no carrossel (subeventos e oficinas) - @claudionsc - @thayanerego
…o de data das suboficinas e subevento - @claudionsc - @thayanerego
…redirecionando para a página específica com o botão saiba mais - @claudionsc - @fernandesmelo
…cription\styled.js): reestruturando descrição de eventos - finalizando estilização da descrição - @claudionsc - @fernandesmelo
…a workshops nas suboficinas dos eventos - @claudionsc - @thayannerego
| @@ -74,96 +116,106 @@ export const EventsPageDescription = () => { | |||
| } | |||
|
|
|||
| return ( | |||
| <div className='full-container'> | |||
| <div className="full-container"> | |||
There was a problem hiding this comment.
A gente está usando aspas simples em todos os lugares, o certo aqui também seria manter o padrão. Foi o eslint que fez isso?
There was a problem hiding this comment.
Isso valeria pro arquivo todo
There was a problem hiding this comment.
Nesse caso, deveríamos colocar as aspas simples? Ficamos na dúvida pois a alteração sugere as aspas duplas, porém percebemos que o padrão são as simples e alteramos para aspas simples
There was a problem hiding this comment.
Não, o certo seria aspas simples. Eu vou ver o motivo de ele tá sugerindo isso
| activitiesList.push(subeventos) | ||
| setActivities(activitiesList) |
There was a problem hiding this comment.
Isso aqui está um pouco estranho, não sei se deveríamos alterar o estado manualmente com push. O React preza por imutabilidade então talvez isso tenha consequências inesperadas. Minha sugestão é usar destructuring ao invés de push:
| activitiesList.push(subeventos) | |
| setActivities(activitiesList) | |
| setActivities([ | |
| ...activitiesList, | |
| ...subeventos | |
| ]) |
Nesse caso, o destructuring cria uma lista nova ao invés de alterar a lista existente, mantendo o comportamento de imutabilidade esperado pelo React.
There was a problem hiding this comment.
Fizemos a mudança, porém para funcionar corretamente foi preciso adicionar a mudança de estado dentro do bloco then e excluir os spreads.
There was a problem hiding this comment.
Vi a versão final agora, tá perfeito! Desde que a gente não altere activities diretamente como estávamos fazendo com push antes.
Entretanto, do jeito que está, estamos criando um array de arrays, certo?
Um array contendo dois arrays, mais precisamente, o primeiro com subeventos e o segundo com oficinas. Não sei se isso é o que vocês queriam fazer mesmo. Se não for, dá para usar concat para deixar tudo num array único. Seria algo mais ou menos assim:
setActivities(subeventos.concat(oficinas))
There was a problem hiding this comment.
Lendo o código agora, estamos fazendo map dentro de map, provavelmente porque temos esse array de arrays, eu acho que o mais eficiente seria ter apenas um array com todos os items de subeventos e oficinas, dado que um subevento e uma oficina têm os mesmos atributos, então é ok eles ficarem num mesmo array.
Acho que seria legal olharmos isso juntos no Discord, essa mudança é um pouco complexa.
…ront into 73/feat-eventos-mostrar-atividades
…ugeridas pelas mentoras - @claudionsc - @thayannerego
…ugeridas pelas mentora - criando helper para formatação de datas - @claudionsc - @thayanneregoo
…om/aceleradora-TW/nossa-casa-front into 73/feat-eventos-mostrar-atividades
…ugeridas pelas mentoras - adicionando helper para formatação de datas - @claudionsc - @thayannerego
| @@ -0,0 +1,35 @@ | |||
| const daysOfWeekInPtBr = [ | |||
There was a problem hiding this comment.
Show de bola! Essa refatoracao ficou otima!
src/helpers/format-data/index.js
Outdated
| const month = date.toLocaleDateString('pt-BR', { month: 'short' }) | ||
| const year = date.toLocaleDateString(undefined, { | ||
| year: 'numeric', | ||
| Timezone: 'UTF' |
There was a problem hiding this comment.
Isso aqui ainda esta errado, o certo seria:
| Timezone: 'UTF' | |
| timeZone: 'UTC' |
… sugeridas pela mentora - fix erro de chave única - @claudionsc - @thayannregoo
… sugeridas pela mentora - fix erro de chave única - @claudionsc - @thayannregoo
…er para formatação de datas - @claudionsc - @thayanneregoo
| @@ -74,96 +116,106 @@ export const EventsPageDescription = () => { | |||
| } | |||
|
|
|||
| return ( | |||
| <div className='full-container'> | |||
| <div className="full-container"> | |||
There was a problem hiding this comment.
Não, o certo seria aspas simples. Eu vou ver o motivo de ele tá sugerindo isso
…ion\styled.js): Resolvendo conflitos com a main - @claudionsc - @joyce-caroline - @Wander06 - @thayanneregoo - @angelones19 - @gabrielapsgomes
…ront into 73/feat-eventos-mostrar-atividades
…escrição para se adequar ao padrão - @claudionsc
#73 - Feat: Eventos: Mostrar atividades
🆙 CHANGELOG