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

add support for Order()->FulfillmentOrder and FulfillmentRequests #278

Merged
merged 3 commits into from
Feb 13, 2023

Conversation

jeroendelau
Copy link
Contributor

@jeroendelau jeroendelau commented Nov 17, 2022

This pull request aims to support the behavior required by:
https://shopify.dev/apps/fulfillment/order-management-apps/manage-fulfillments#request-a-fulfillment
https://shopify.dev/api/admin-rest/2022-10/resources/assignedfulfillmentorder

  • FulfillmentRequest Resource - including actions
  • Mapped FulfillmentOrder->FulfillmentRequest
  • Mapped Order(id)->FulfillmentOrder

Example:

// Requesting the FulfilmentOrder for a given order
$fo = $client->Order("1234567890")->FulfillmentOrder()->get();

// Creating a FulfilmentRequest
// Follow instructions to get partial fulfilments
$fr = $client->FulfillmentOrder('0987654321')->FulfillmentRequest->post([]);

// Accepting \ Rejecting a FulfilmentRequest
$fr = $client->FulfillmentOrder('0987654321')->FulfillmentRequest->accept();
$fr = $client->FulfillmentOrder('0987654321')->FulfillmentRequest->reject();

// Communicating fulfillment
$client->Fulfillment->post($body)

@jeroendelau
Copy link
Contributor Author

jeroendelau commented Dec 19, 2022

@tareqtms is there something holding up this merge? Anything needs changing or explaining?

@tareqtms
Copy link
Contributor

@jeroendelau
Sorry for the delay. I just need to add the examples somewhere in the readme file. It would be good if you can do it, otherwise I will do and merge thereafter.

@jeroendelau
Copy link
Contributor Author

@tareqtms I added the docs

@Coennie
Copy link

Coennie commented Feb 9, 2023

Ah this is what I'm looking for in #288
Could it be merged asap?

@tareqtms
Copy link
Contributor

tareqtms commented Feb 9, 2023

@Coennie Yeah, I will merge it today or tomorrow.

@Coennie
Copy link

Coennie commented Feb 10, 2023

@Coennie Yeah, I will merge it today or tomorrow.

hi @tareqtms really hope you can make some time to merge this. Regards

@tareqtms tareqtms merged commit d5759b3 into phpclassic:master Feb 13, 2023
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