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(museu): add link to museum #433

Closed
wants to merge 2 commits into from
Closed

feat(museu): add link to museum #433

wants to merge 2 commits into from

Conversation

rodrigoKulb
Copy link
Contributor

@rodrigoKulb rodrigoKulb commented Jun 3, 2022

@filipedeschamps aqui apenas uma sugestão para adicionar o link para página Museu, como é bem simples já criei a PR. Super tranquilo se não achar uma boa ideia.

Criei a PR que era mais simples que criar uma Issue para perguntar!

Ótimo final de semana a todos!

Aqui uma sugestão para adicionar link a página museu, também adicionei a opção de voltar ao portal
Tabnews
@vercel
Copy link

vercel bot commented Jun 3, 2022

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

To accomplish this, @rodrigoKulb needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

@filipedeschamps
Copy link
Owner

Criei a PR que era mais simples que criar uma Issue para perguntar!

😂 sensacional! E vou colar aqui um comentário que acabei de fazer em outro PR que não está na Milestone:

Como coloquei em um outro PR, quero tentar focar ao máximo nas issues da Milestone 4 para conseguir chegar nas TabCoins, e o nível de dificuldade começa aumentar então o foco e esforço também 🤝

Mas vamos deixar todos esses PRs de melhoria abertos para começar a fazer merge deles no final da Milestone 👍

De qualquer forma, não sugiro levar o usuário do menu principal para uma página tão diferente (apesar de você ter adicionado um link para voltar). Vejo isso mais num espaço tipo o footer da página e que hoje não existe.

@rodrigoKulb
Copy link
Contributor Author

@filipedeschamps fechado concordo, ideal seria dentro de um footer, sobre a #345 tentei achar algo para ajudar, mas não sei se consegue separar / dividir mais as atividades necessárias, assim posso tentar fazer algo por lá!

@rodrigoKulb rodrigoKulb closed this Jun 3, 2022
@rodrigoKulb rodrigoKulb deleted the link-museu branch June 3, 2022 15:07
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