Skip to content

Fix/api 13/pagination#8

Merged
andres15alvarez merged 6 commits into
devfrom
fix/API-13/pagination
Mar 9, 2025
Merged

Fix/api 13/pagination#8
andres15alvarez merged 6 commits into
devfrom
fix/API-13/pagination

Conversation

@iabrahaamxs
Copy link
Copy Markdown
Contributor

image

Copy link
Copy Markdown
Member

@andres15alvarez andres15alvarez left a comment

Choose a reason for hiding this comment

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

por favor @CarlosDev73 o @Dazzlin00 realice alguno tambien code review para aportar otro punto de vista, a partir de ahora si usaremos el double check

Comment thread src/utils/dto.ts Outdated
Comment thread src/utils/paginationUrls.ts Outdated
Comment thread src/utils/paginationUrls.ts Outdated
Comment thread src/utils/paginationUrls.ts Outdated
Comment thread src/products/products.service.ts
Copy link
Copy Markdown
Member

@andres15alvarez andres15alvarez left a comment

Choose a reason for hiding this comment

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

lgtm

@andres15alvarez
Copy link
Copy Markdown
Member

@CarlosDev73 @Dazzlin00 esperamos por su code review

@andres15alvarez andres15alvarez merged commit 798574a into dev Mar 9, 2025
@andres15alvarez andres15alvarez deleted the fix/API-13/pagination branch March 9, 2025 21:41
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.

3 participants