Skip to content

Rubricas dinâmicas#590

Merged
joellensilva merged 3 commits into
mainfrom
rubricas-din
May 5, 2025
Merged

Rubricas dinâmicas#590
joellensilva merged 3 commits into
mainfrom
rubricas-din

Conversation

@joellensilva

@joellensilva joellensilva commented Apr 24, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • Depende retornando as rubricas de forma dinâmica storage#145
  • as rubricas eram retornadas uma a uma
  • com esta modificação, é feita uma consulta no banco para retornar todas as rubricas
  • além disso, o golang é baseado em structs, então substituímos por um dicionário para atender as necessidades

@JezzDiego JezzDiego mentioned this pull request Apr 24, 2025
Merged
@joellensilva joellensilva marked this pull request as ready for review April 28, 2025 18:07
rdurl0
rdurl0 approved these changes Apr 29, 2025
View reviewed changes

@rdurl0 rdurl0 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

show!

JezzDiego
JezzDiego approved these changes May 5, 2025
View reviewed changes

@JezzDiego JezzDiego left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@joellensilva joellensilva merged commit a2a4617 into main May 5, 2025
2 checks passed
@joellensilva joellensilva deleted the rubricas-din branch May 5, 2025 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rdurl0 rdurl0 rdurl0 approved these changes

@JezzDiego JezzDiego JezzDiego approved these changes

@talitalobo talitalobo Awaiting requested review from talitalobo

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@joellensilva @rdurl0 @JezzDiego