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

refactor: add suggestion to fetch sessions #38

Open
wants to merge 1 commit into
base: activity
Choose a base branch
from

Conversation

Azanniel
Copy link
Collaborator

Descrição

Sugestões de modificações para a atividade da @a-mandaflores.

Segue algumas modificações que talvez possam auxiliar no desenvolvimento da sua atividade, fico contente em poder ajudar e qualquer dúvida posso também participar nas revisões.

Mudanças realizadas

  • Renomeada função e arquivo para getSessionsByDay para ficar semanticamente compreensível
  • Modificação das tipagens utilizadas para estar de acordo com a rota fornecida pela api /session
  • Feito exemplo de uso com server components com a estratégia de URL State (Search Params)

Tipo de mudança 🏗️

Minha mudança é uma:

  • Correção de bug (alteração que corrige um problema)
  • Novo recurso (alteração que adiciona funcionalidades)
  • Esta alteração requer uma atualização de documentação

@Azanniel Azanniel requested a review from a-mandaflores July 25, 2024 16:30
@Azanniel Azanniel self-assigned this Jul 25, 2024
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.

1 participant