Skip to content
This repository was archived by the owner on Oct 6, 2026. It is now read-only.

[FIX] Fix Gift Card Adjustment - #605

Closed
ngi-odoo wants to merge 1 commit into
Shopify:mainfrom
ngi-odoo:patch-1
Closed

ngi-odoo wants to merge 1 commit into
Shopify:mainfrom
ngi-odoo:patch-1

Conversation

@ngi-odoo

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

Copy link
Copy Markdown

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

ngi-odoo commented Sep 7, 2022 •

Copy link
Copy Markdown
Author

CLA signed!

@ngi-odoo

ngi-odoo commented Dec 1, 2022

Copy link
Copy Markdown
Author

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

@ngi-odoo

ngi-odoo commented Dec 1, 2022

Copy link
Copy Markdown
Author

@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

@Suleimanlatrsh

Copy link
Copy Markdown
Contributor

Thanks for contributing! This package is deprecated and we're archiving this repo, so we're closing all open issues and PRs. Please use shopify-app-python instead, and open new issues there. More context: Rethinking support for PHP/Python packages

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants