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

feat: Adding github project link #54

Merged
merged 2 commits into from
May 18, 2024

Conversation

marcieltorres
Copy link
Contributor

@marcieltorres marcieltorres commented May 17, 2024

A ideia dessa alteração é divulgar também no site da iniciativa que o projeto está com o código no github, fiz isso pensando muito na minha própria utilização também, pois eu já havia esbarrado com o site e não sabia que o código estava aberto, com isso acredito que outras pessoas também possam saber do projeto e quem sabe contribuir também.

About page

Não fui tão criativo aqui nessa página, então se vcs tiverem uma sugestão de texto melhor ❤️

image

footer

image

Mobile

image

image

Copy link

vercel bot commented May 17, 2024

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

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
sos-pet ⬜️ Ignored (Inspect) Visit Preview May 18, 2024 5:26pm

@emiliosheinz
Copy link
Owner

Obrigado pela contribuição @marcieltorres, você consegue por favor anexar um print de como fica no mobile? Agradeço!

@marcieltorres
Copy link
Contributor Author

Obrigado pela contribuição @marcieltorres, você consegue por favor anexar um print de como fica no mobile? Agradeço!

image

Editei a descrição do pr tbm com os prints 🚀

@emiliosheinz
Copy link
Owner

Vou verificar o motivo pelo qual tem os Checks não estão passando. Parece alguma config relacionada ao repositório.

@marcieltorres
Copy link
Contributor Author

Vou verificar o motivo pelo qual tem os Checks não estão passando. Parece alguma config relacionada ao repositório.

Acho que tem que injetar as env vars não tem?

@emiliosheinz
Copy link
Owner

Acho que tem que injetar as env vars não tem?

Sim, mas por algum motivo só no teu PR está falhando. Tenho que entender porque!

@marcieltorres
Copy link
Contributor Author

marcieltorres commented May 17, 2024

Acho que tem que injetar as env vars não tem?

Sim, mas por algum motivo só no teu PR está falhando. Tenho que entender porque!

Eu rodei o lint local e foi de boa, e olhando na action tá criando o .env né https://github.com/emiliosheinz/sos-pet/blob/dev/.github/workflows/lint-type-check-and-build.yaml#L23

@emiliosheinz
Copy link
Owner

@marcieltorres consegue atualizar a tua branch com o que está na dev? Acabei de dar merge #55 que corrige o problema que estamos enfrentando neste PR.

@emiliosheinz emiliosheinz merged commit b34b1ad into emiliosheinz:dev May 18, 2024
3 checks passed
@marcieltorres marcieltorres deleted the feat/github-project-link branch May 18, 2024 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants