Skip to content

Conversation

@mamunozcar
Copy link
Contributor

@mamunozcar mamunozcar commented Nov 7, 2025

Description

This PR add the resource PaymentLink with its corresponding manager.

Also, it adds the check-elegibility endpoint for the PaymentIntents resource

Requirements

None.

Additional changes

It was necessary to modify the resource_update, because it did not allow custom routes, it only allowed routes with the defined ID.

@mamunozcar mamunozcar marked this pull request as ready for review November 7, 2025 15:06
@mamunozcar mamunozcar requested review from Gonzalo9823, mari-hernandez, parayab and rudyjb24 and removed request for parayab and rudyjb24 November 7, 2025 15:17
Copy link
Contributor

@Gonzalo9823 Gonzalo9823 left a comment

Choose a reason for hiding this comment

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

LGTM! Deje un comentario chico nomas!

@mamunozcar mamunozcar force-pushed the feat-add-payment-links-and-check-elegibility-endpoint branch from c7c652a to f744e45 Compare November 7, 2025 15:34
Copy link

@mari-hernandez mari-hernandez left a comment

Choose a reason for hiding this comment

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

Lo veo bien!

Copy link

@mari-hernandez mari-hernandez left a comment

Choose a reason for hiding this comment

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

lo veo bien :)

Creo que estaría bueno para la proxima que dejemos la referencia de los docs para los endpoints que estamos agregando, así queda mas facil revisar

@mamunozcar mamunozcar merged commit 533b0a5 into master Nov 10, 2025
2 checks passed
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.

4 participants