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

Criar página sobre a evolução do TabNews (da concepção ao lançamento) no museu #1582

Merged
merged 2 commits into from
Jan 4, 2024

Conversation

Rafatcb
Copy link
Collaborator

@Rafatcb Rafatcb commented Dec 24, 2023

Mudanças realizadas

Esse PR é uma continuação do PR #946, realizando modificações conforme comentários. A intenção da página é mostrar, de forma impessoal, a evolução do TabNews desde o surgimento da ideia até o dia do lançamento. Tirei alguns poucos detalhes do outro PR. Talvez mais pudessem ser tirados. Adicionei mais links de referência ao longo do texto.

Resolvi criar uma página no próprio estilo do site porque, apesar de ser do Museu, não há necessidade de ter estilização própria, já que ela conta uma história ao invés de ser a própria história, como são as sugestões de design do TabNews.

Fiz com Markdown pelo mesmo motivo mencionado no FAQ (padrão), mas como é uma página do Museu, não vejo problema em continuar como Markdown. Será uma página bem menos acessada e não temos necessidade de âncoras para os títulos, por exemplo.

As imagens não estarem hospedadas no próprio TabNews pode ser uma preocupação, mas pensei em colocar no Wayback Machine e outros arquivadores assim que o PR for mergeado, o que possibilitará que tenhamos um backup das imagens em caso de necessidade. Também coloquei um alt descritivo em todas as imagens.

Tipo de mudança

  • Nova funcionalidade

Checklist:

  • As modificações não geram novos logs de erro ou aviso (warning).
  • Eu adicionei testes que provam que a correção ou novo recurso funciona conforme esperado.
  • Tanto os novos testes quanto os antigos estão passando localmente.

@Rafatcb Rafatcb added front Envolve modificações no frontend novo recurso Nova funcionalidade/recurso labels Dec 24, 2023
@Rafatcb Rafatcb self-assigned this Dec 24, 2023
Copy link

vercel bot commented Dec 24, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tabnews ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 4, 2024 0:50am

Copy link
Collaborator

@aprendendofelipe aprendendofelipe left a comment

Choose a reason for hiding this comment

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

Show @Rafatcb! 💪

Quando eu sugeri (#946 (comment)) deixar o texto mais impessoal, só estava falando sobre o estilo do texto, que ficou melhor agora, mas não que o autor deveria ser ocultado.

Então poderia deixar a informação de que o @gabrielsozinho foi o autor e o @Rafatcb o revisor. Isso pode ficar logo abaixo do título. O que acham?

Uma coisa que me preocupa é se um dia perdermos imagens tão legais que estão no artigo, mas hospedadas fora do nosso controle, em imgur.com e user-images.githubusercontent.com. Acham que teria como recuperar, ou é melhor já garantir e deixar sob nossa responsabilidade?

pages/museu/index.public.js Outdated Show resolved Hide resolved
@Rafatcb
Copy link
Collaborator Author

Rafatcb commented Jan 4, 2024

Então poderia deixar a informação de que o @gabrielsozinho foi o autor e o @Rafatcb o revisor. Isso pode ficar logo abaixo do título. O que acham?

Coloquei o Gabriel como autor na página do Museu, e ele como autor e eu revisor no artigo. Veja se era isso que você pensou. Se estiver tudo certo, faço um squash após sua aprovação.

Aproveitei para usar as tags article e address no artigo, e também tirei o estilo da lista para não ficar destoante do resto do site.

Museu Artigo
Print do Museu Print do Artigo

Uma coisa que me preocupa é se um dia perdermos imagens tão legais que estão no artigo, mas hospedadas fora do nosso controle, em imgur.com e user-images.githubusercontent.com. Acham que teria como recuperar, ou é melhor já garantir e deixar sob nossa responsabilidade?

Eu pensei nisso enquanto fazia, não sei se você chegou a ver na descrição do PR:

As imagens não estarem hospedadas no próprio TabNews pode ser uma preocupação, mas pensei em colocar no Wayback Machine e outros arquivadores assim que o PR for mergeado, o que possibilitará que tenhamos um backup das imagens em caso de necessidade.

Realmente não tem onde deixar as imagens de forma segura atualmente, mas acho que os arquivadores já serão um bom fallback para obter as imagens novamente caso o Imgur ou GitHub apague. Aproveitei para arquivar agora com o deploy do PR em homologação:

@aprendendofelipe
Copy link
Collaborator

Coloquei o Gabriel como autor na página do Museu, e ele como autor e eu revisor no artigo. Veja se era isso que você pensou.

Perfeito!

Se estiver tudo certo, faço um squash após sua aprovação.

Você está deixando os commits separados para facilitar a visualização do que foi alterado em cada passo da revisão? Se for isso, não é necessário, pois, mesmo substituindo o commit com um git push -f, é só clicar em Compare na timeline. Por exemplo:

image

Quando achar que o PR pode estar pronto, é melhor já fazer o squash, tanto para permitir o merge após a revisão, mas principalmente para não zerar a lista dos revisores da versão final.

Só para confirmar se não estou falando besteira, vamos reparar o que vai ocorrer com a minha aprovação após você fazer o squash, pois é o que iria ocorrer com qualquer outro contribuidor que tiver revisado seu código. Agora está assim:

image

Aproveitei para usar as tags article e address no artigo, e também tirei o estilo da lista para não ficar destoante do resto do site.

Show!

Eu pensei nisso enquanto fazia, não sei se você chegou a ver na descrição do PR:

Comi bola nesse ponto.

Realmente não tem onde deixar as imagens de forma segura atualmente, mas acho que os arquivadores já serão um bom fallback para obter as imagens novamente caso o Imgur ou GitHub apague. Aproveitei para arquivar agora com o deploy do PR em homologação

Maravilha!

@Rafatcb Rafatcb force-pushed the create-museum-tabnews-evolution branch from f79afa8 to 8fc4131 Compare January 4, 2024 12:02
@Rafatcb
Copy link
Collaborator Author

Rafatcb commented Jan 4, 2024

@aprendendofelipe Não tinha percebido que dava para ver as mudanças dessa forma, mesmo com squash.

Deixei os commits do Gabriel Sozinho e o meu separados.

@aprendendofelipe
Copy link
Collaborator

@aprendendofelipe Não tinha percebido que dava para ver as mudanças dessa forma, mesmo com squash.

Legal que não mudou o status de aprovação, provavelmente porque não mudou nada no código 🎉

image

Será que vai mudar o status após você fazer rebase? Vale o mesmo que falei sobre o squash, sempre que o código estiver "pronto", já faz um rebase. 🤝

Deixei os commits do Gabriel Sozinho e o meu separados.

Show!

@Rafatcb Rafatcb force-pushed the create-museum-tabnews-evolution branch 3 times, most recently from be982e0 to b970a90 Compare January 4, 2024 12:38
@Rafatcb
Copy link
Collaborator Author

Rafatcb commented Jan 4, 2024

Fiz algumas tentativas mas não consegui juntar o merge e o meu commit sem parecer que eu modifiquei os outros arquivos também. Nunca usei o rebase (que foi a minha primeira tentativa e acabou duplicando commits), vou tentar arrumar de novo mais tarde.

@Rafatcb Rafatcb force-pushed the create-museum-tabnews-evolution branch from 2d7878f to 35909fe Compare January 4, 2024 12:48
@Rafatcb
Copy link
Collaborator Author

Rafatcb commented Jan 4, 2024

Depois de algumas tentativas, parece que foi 😅

@aprendendofelipe
Copy link
Collaborator

Fiz algumas tentativas mas não consegui juntar o merge e o meu commit sem parecer que eu modifiquei os outros arquivos também. Nunca usei o rebase (que foi a minha primeira tentativa e acabou duplicando commits), vou tentar arrumar de novo mais tarde.

Com o PR pronto, acho que é mais fácil fazer diretamente pela interface do GitHub:

image

Só que, se eu fizer, fico como coator dos commits, por isso é legal o próprio autor do PR fazer.


E sobre o nosso teste, veja que mesmo sem mudança no código, sumiu a aprovação:

image

@aprendendofelipe aprendendofelipe merged commit a144545 into main Jan 4, 2024
7 checks passed
@aprendendofelipe aprendendofelipe deleted the create-museum-tabnews-evolution branch January 4, 2024 13:31
@aprendendofelipe
Copy link
Collaborator

@Rafatcb e @gabrielsozinho, o artigo já está disponível em produção 🚀🚀🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
front Envolve modificações no frontend novo recurso Nova funcionalidade/recurso
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants