Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

SDKs-PHP for cards #240

Merged
merged 9 commits into from
Oct 7, 2021
Merged

SDKs-PHP for cards #240

merged 9 commits into from
Oct 7, 2021

Conversation

ankitdas13
Copy link
Contributor

card module with update README

README.md Outdated

// Customers
$customer = $api->customer->create(array('name' => 'Razorpay User', 'email' => 'customer@razorpay.com')); // Creates customer
$customer = $api->customer->fetch($customerId); // Returns a particular customer
$customer = $api->customer->edit(array('name' => 'Razorpay User', 'email' => 'customer@razorpay.com')); // Edits customer

// Tokens
$token = $api->customer->token()->fetch($tokenId); // Returns a particular token
$token = $api->customer->fetch($customerId)->tokens()->fetch($tokenId); // Returns a particular token
$tokens = $api->customer->token()->all($options); // Returns array of token objects
Copy link
Contributor

Choose a reason for hiding this comment

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

Corret this $api->customer->token()->all($options) also

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i update this function and add seprate comments for card module

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i update this function and there is no need to pass optional param
https://razorpay.com/docs/api/recurring-payments/cards/tokens/#22-fetch-tokens-by-customer-id

Copy link
Contributor

@HancilSequeira HancilSequeira left a comment

Choose a reason for hiding this comment

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

LGTM

@neera11 neera11 merged commit ce7e2e3 into master Oct 7, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants