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

Feat/hold fulfillment order #660

Merged
merged 5 commits into from
Aug 20, 2024

Conversation

aldochaconc
Copy link
Contributor

No description provided.

@aldochaconc aldochaconc reopened this Aug 14, 2024
resources/fulfillment-order.js Outdated Show resolved Hide resolved
resources/fulfillment-order.js Outdated Show resolved Hide resolved
types/index.d.ts Outdated Show resolved Hide resolved
types/index.d.ts Outdated Show resolved Hide resolved
types/index.d.ts Outdated Show resolved Hide resolved
types/index.d.ts Outdated Show resolved Hide resolved
@lpinca
Copy link
Collaborator

lpinca commented Aug 14, 2024

Please add back the test and update the README. Also, if this lands, https://shopify.dev/docs/api/admin-rest/2024-07/resources/fulfillmentorder#post-fulfillment-orders-fulfillment-order-id-release-hold should also be added.

types/index.d.ts Outdated Show resolved Hide resolved
@coveralls
Copy link

coveralls commented Aug 14, 2024

Coverage Status

coverage: 100.0%. remained the same
when pulling 749760f on aldochaconc:feat/hold-fulfillment-order
into 6080d51 on MONEI:master.

@lpinca lpinca force-pushed the feat/hold-fulfillment-order branch from 3ff6830 to 749760f Compare August 20, 2024 09:31
@lpinca lpinca merged commit 7803fa1 into MONEI:master Aug 20, 2024
9 checks passed
@lpinca
Copy link
Collaborator

lpinca commented Aug 20, 2024

Thank you.

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.

3 participants