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

[FIX] Fix Gift Card Adjustment #605

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

ngi-odoo
Copy link

@ngi-odoo ngi-odoo commented Sep 7, 2022

WHY are these changes introduced?

According to the api (https://shopify.dev/api/admin-rest/2022-07/resources/gift-card-adjustment#get-gift-cards-gift-card-id-adjustments), the url should not include /admin before /gift_cards. This commit removes the /admin in order to make it working

WHAT is this pull request doing?

Remove the /admin path from the prefix source of GiftCardAdjustment

Checklist

  • I have updated the CHANGELOG (if applicable)
  • I have followed the Shopify Python guide

According to the api (https://shopify.dev/api/admin-rest/2022-07/resources/gift-card-adjustment#get-gift-cards-gift-card-id-adjustments), the url should not include /admin before /gift_cards.
This commit removes the /admin in order to make it working
@ngi-odoo
Copy link
Author

ngi-odoo commented Sep 7, 2022

CLA signed!

@ngi-odoo
Copy link
Author

ngi-odoo commented Dec 1, 2022

Looks like the bot doesn't detect I've signed the CLA

@ngi-odoo
Copy link
Author

ngi-odoo commented Dec 1, 2022

@andyw8 I don't know if you are the right person to ping to check this PR, if not is it possible to ping the correct one?
I've signed the CLA but the github bot haven't detected it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant