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

[FEATURE] modify the operation macro to accept PaymentData as argument #6434

Open
2 tasks done
Narayanbhat166 opened this issue Oct 25, 2024 · 0 comments
Open
2 tasks done
Assignees
Labels
C-feature Category: Feature request or enhancement S-awaiting-triage Status: New issues that have not been assessed yet

Comments

@Narayanbhat166
Copy link
Member

Feature Description

#[derive(Debug, Clone, Copy, PaymentOperation)]
#[operation(operations = "all", flow = "authorize")]
pub struct PaymentConfirm;

The current operation macro does not accept paymentData in the arguments. Since each operation can have a different payment data now which was introduced in this PR.

The operations macro should also be modified to accept or use a different PaymentData internally when deriving the Operations trait.

Possible Implementation

NA

Have you spent some time checking if this feature request has been raised before?

  • I checked and didn't find a similar issue

Have you read the Contributing Guidelines?

Are you willing to submit a PR?

No, but I'm happy to collaborate on a PR with someone else

@Narayanbhat166 Narayanbhat166 added C-feature Category: Feature request or enhancement S-awaiting-triage Status: New issues that have not been assessed yet labels Oct 25, 2024
@AnuthaDev AnuthaDev self-assigned this Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-feature Category: Feature request or enhancement S-awaiting-triage Status: New issues that have not been assessed yet
Projects
None yet
Development

No branches or pull requests

2 participants