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

adiciona qd nas universidades base de conhecimento #261

Merged
merged 5 commits into from
Apr 15, 2024

Conversation

kaiocp
Copy link

@kaiocp kaiocp commented Apr 14, 2024

Português (BR) | English (US)

Comunidade

Tipo de alteração

  • 🐞 Correção de problema
  • ✨ Melhoria ou nova funcionalidade
  • 📰 Nova postagem no blog

Issues relacionadas

Issues que são relacionadas a esta Pull Request:

#247

Validação

  • Validei a alteração no link gerado pelo bot da Netlify (Deploy Preview/Preview on mobile)
  • Validei o Layout responsivo (desktop/mobile) após a implementação
  • Verifiquei o registro do deploy (Latest deploy log) e nenhum novo alerta ou erro foi adicionado

Evidências

mobile

Screen Recording 2024-04-15 at 17 58 27

desktop

Screen Recording 2024-04-15 at 17 57 53

Documentação

  • A documentação deste repositório foi atualizada (quando necessário).
  • Esta alteração requer que a documentação externa seja atualizada.

Copy link
Member

@ogecece ogecece left a comment

Choose a reason for hiding this comment

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

@kaiocp meu único comentário aqui seria uma coisa que deixei passar no último PR:

A ordem das páginas na navbar / navmenu estão meio diferentes.

Mobile:
image

Desktop:
image

O "Universidades" deveria vir antes do "Sobre" no mobile

@kaiocp
Copy link
Author

kaiocp commented Apr 15, 2024

A ordem das páginas na navbar / navmenu estão meio diferentes

nice catch, @ogecece! consertado em c58380e

preview Screenshot 2024-04-15 at 18 02 04

@kaiocp kaiocp requested a review from ogecece April 15, 2024 21:01
@ogecece ogecece force-pushed the kaiocp/qd-uni-knowledge-base branch from c58380e to b7db932 Compare April 15, 2024 21:25
Copy link
Member

@ogecece ogecece left a comment

Choose a reason for hiding this comment

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

Boa @kaiocp !

@ogecece ogecece merged commit 5936bda into okfn-brasil:dev-qd-uni Apr 15, 2024
@kaiocp kaiocp deleted the kaiocp/qd-uni-knowledge-base branch April 15, 2024 22:18
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