tabnews.com.br icon indicating copy to clipboard operation
tabnews.com.br copied to clipboard

feat(evolution-of-tabnews): add evolution of TabNews article to museum page

Open gabrielsozinho opened this issue 2 years ago • 7 comments

Conforme sugerido pelo Filipe neste comentário. Eu não sei se era essa a ideia inicial, mas se não for, só avisar pra mim, que faço as alterações necessárias. 🤙

gabrielsozinho avatar Nov 24 '22 15:11 gabrielsozinho

@gabrielsozinho is attempting to deploy a commit to the TabNews Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Nov 24 '22 15:11 vercel[bot]

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

Name Status Preview Updated
tabnews ✅ Ready (Inspect) Visit Preview Nov 24, 2022 at 4:16PM (UTC)

vercel[bot] avatar Nov 24 '22 16:11 vercel[bot]

Hmm, porque do nada a vercel fez o deploy e o github rodou os testes?

33gustavo33 avatar Nov 24 '22 16:11 33gustavo33

Hmm, porque do nada a vercel fez o deploy e o github rodou os testes?

Foi liberado por um membro do time na Vercel 😉

aprendendofelipe avatar Nov 24 '22 16:11 aprendendofelipe

@Rafatcb obrigado pelos toques! Commitei as alterações, está tudo certo?

gabrielsozinho avatar Nov 24 '22 17:11 gabrielsozinho

@Rafatcb obrigado pelos toques! Commitei as alterações, está tudo certo?

Você corrigiu com Find and Replace? Alguns Idéia perderam o case, ficando ideia ao invés de Ideia 😅

Rafatcb avatar Nov 24 '22 18:11 Rafatcb

Você corrigiu com Find and Replace? Alguns Idéia perderam o case, ficando ideia ao invés de Ideia 😅

Caramba, foi exatamente isso. Nem passou pela minha cabeça que isso iria ocorrer.

gabrielsozinho avatar Nov 24 '22 18:11 gabrielsozinho

Da abertura desse PR até hoje, muita coisa mudou no TabNews. Para não precisar ser uma página que é atualizada constantemente, e para continuar fazendo sentido, acho que podemos atualizá-la para "A Evolução do TabNews — da Concepção ao Lançamento" ou algo parecido.

Boa! 👍

Assim, fica em aberto também a possibilidade de incluir mais detalhes em outro PR, como os detalhes que o Filipe comentou.

Isso provavelmente será outro item no museu, mas me levou a pensar se o artigo desse PR não deveria ser um pouco mais impessoal para estar mais aberto a atualizações ou correções futuras. O que acham?


O que acham de hospedar as imagens na pasta public? Eu já vi issues em que as imagens não estão mais aparecendo, mesmo sendo hospedadas pelo próprio GitHub.

aprendendofelipe avatar Dec 18 '23 11:12 aprendendofelipe

Isso provavelmente será outro item no museu, mas me levou a pensar se o artigo desse PR não deveria ser um pouco mais impessoal para estar mais aberto a atualizações ou correções futuras. O que acham?

Eu estava pensando nisso. Na época que eu criei o PR eu pensava que a sugestão do Filipe tinha sido essa (de como está hoje), mas hoje isso não faz tanto sentido.

Também não estou tão disponível para fazer os ajustes necessários, então acho melhor alguém criar um novo PR com os ajustes necessários, não faço questão alguma dos "créditos". (Obs.: o mesmo se aplica ao PR #1210, que está até mais cru que esse aqui, então é ainda melhor outra pessoa criar um novo PR). 👍

gabrielsozinho avatar Dec 18 '23 12:12 gabrielsozinho

Obs.: o mesmo se aplica ao PR https://github.com/filipedeschamps/tabnews.com.br/pull/1210, que está até mais cru que esse aqui, então é ainda melhor outra pessoa criar um novo PR

Eu posso criar um novo PR para o FAQ, tomando como base o seu e os comentários que fiz conforme analisava, só não sei se vou conseguir te deixar como contribuidor nos commits (me parece que o @aprendendofelipe chegou a fazer isso uma vez, mas ainda não parei para entender ao certo como funcionou).

Rafatcb avatar Dec 18 '23 14:12 Rafatcb

Eu posso criar um novo PR para o FAQ, tomando como base o seu e os comentários que fiz conforme analisava, só não sei se vou conseguir te deixar como contribuidor nos commits (me parece que o @aprendendofelipe chegou a fazer isso uma vez, mas ainda não parei para entender ao certo como funcionou).

Para deixar o @gabrielsozinho como coautor de qualquer commit, basta adicionar Co-authored-by: Gabriel Sozinho <[email protected]> na última linha da mensagem do commit. Você pode pegar esse dado em qualquer commit dele fazendo um git show <commit>.

Se for o caso de usar um commit específico dele, sem alterações, aí não fica como coautor, mas sim com a autoria original dele. Nesse caso é só usar o git cherry-pick <commit> para importar determinado commit na branch que estiver trabalhando.

Se um PR contém todos os commits de outro PR (não tenho certeza se o outro precisa estar aberto), então ao fazer merge de um, o outro também aparece como mesclado. Mas acredito que não vai ser o caso aqui, pois precisaria organizar os commits primeiro.

aprendendofelipe avatar Dec 18 '23 14:12 aprendendofelipe

Gabriel, obrigado pela colaboração! Esse PR está sendo continuado em https://github.com/filipedeschamps/tabnews.com.br/pull/1582, com as modificações propostas nos comentários.

Rafatcb avatar Dec 24 '23 00:12 Rafatcb