search

okfn-brasil / querido-diario-frontend

Repositório com a implementação do frontend da Plataforma de Busca do Querido Diário
https://queridodiario.ok.org.br
MIT License
36 stars 32 forks source link

Implementação de funcionalidades de download de excertos selecionados na página geral e no Portal Educação #262

Closed csafurtado closed 3 months ago

csafurtado commented 4 months ago

Português (BR) | English (US)

Comunidade

Tipo de alteração

Issues relacionadas

Até o momento, sem issues relacionadas.

Validação

Evidências

Anexe evidências do antes e do depois da alteração (quando necessário).

Documentação

netlify[bot] commented 4 months ago

Deploy Preview for querido-diario-plataforma ready!

Name Link
Latest commit a75daeed777dfb3003111f06a4ea68ee01c7bf84
Latest deploy log https://app.netlify.com/sites/querido-diario-plataforma/deploys/66630cd62772720008430a6a
Deploy Preview https://deploy-preview-262--querido-diario-plataforma.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

csafurtado commented 4 months ago

@ogecece as modificações solicitadas foram realizadas!

ogecece commented 4 months ago

Validei as outras funcionalidades. Só aguardando os commits serem substituídos para poder iniciar a revisão de código!

wildemberg-sales commented 4 months ago

Foi efetuado a correção dos problemas apontados, e corrigido o problema com as ramificações, atualmente a árvore de commits está linear e sem duplicações

ArthurFerreiraRodrigues commented 4 months ago

Primeiro, desculpem pelo número de pedidos de alteração. Como a busca do QD e a do QD-Edu tem suas diferenças, tentei encontrar maneiras de tornar pelo menos as nomenclaturas consistentes e fui apontando onde conseguia. Posso ter deixado coisas passarem e já peço desculpas se uma próxima revisão de código tenha pedidos que deixei passar nessa primeira.

Segunda coisa é que agora que consegui fazer o review de código tenho um pedido. Assim que a gente completar as modificações e disser que o código está aprovado, podemos fazer squash em todos os commits de "correção"? Só pra manter os commits que realmente estão adicionando alguma funcionalidade ou implementando uma mudança de estilo.

Pode ser feito esse squash sim. É o ideal.

Eu fiz um commit vazio enquanto tentava remover uma linha extra, então perdão pelo vacilo. Fora ele, estou tentando condensar ao máximo os commits.

ArthurFerreiraRodrigues commented 4 months ago

@ogecece Correções e refatorações aplicadas. Solicito sua revisão.

csafurtado commented 3 months ago

As modificações solicitadas foram aplicadas @ogecece! Poderia verificá-las por gentileza?

ogecece commented 3 months ago

Pode fazer os squashs nos commits de correção

csafurtado commented 3 months ago

@ogecece o squash no caso você mesmo pode fazer na própria opção de dar o merge aqui. Seria esse squash que vc se refere?

ogecece commented 3 months ago

Esse daqui seria pra reduzir todos os commits do PR a um commit apenas. O que eu queria é que vocês usassem o rebase interativo para fazer squash apenas nos commits de correção e manter os commits de funcionalidade

csafurtado commented 3 months ago

Aah sim entendi! Faremos então!

ogecece commented 3 months ago

Squash aprovado!