Skip to content

[Junie]: Update Shopee OpenAPI for Returns, Product, and SBS - #50

Merged
nVuln merged 1 commit into
masterfrom
junie/issue-49-31854268960
Aug 28, 2026
Merged

nVuln merged 1 commit into
masterfrom
junie/issue-49-31854268960

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

📌 Hey! This PR was made for you with Junie, the coding agent by JetBrains Early Access Preview

It's still learning, developing, and might make mistakes. Please make sure you review the changes before you accept them.
We'd love your feedback — join our Discord to share bugs, ideas: here.

📊 Junie Summary:

Summary

  • Implemented the only major API-surface change from announcement #1528 that affects this SDK: added support for the new endpoint v2.sbs.get_fulfillment_mapping_inventory_list.
  • Ignored return/product field/validation changes as requested (non-major parameter/field-level updates).

Changes

  • Added new resource EcomPHP\Shopee\Resources\Sbs with method getFulfillmentMappingInventoryList($params = []), mapped to GET sbs/get_fulfillment_mapping_inventory_list.
  • Wired the new resource into Client (use import, @property-read Sbs $Sbs, and resource registry entry) so it is accessible via $client->Sbs.
  • Added tests: tests/Resources/SbsTest.php (endpoint path/query mapping + default page_size) and tests/ClientTest.php::testSbsResourceAccess.

Verification

  • Ran: vendor/bin/phpunit tests/Resources/SbsTest.php tests/ClientTest.php --filter "(testGetFulfillmentMappingInventoryList|testGetFulfillmentMappingInventoryListWithDefaultParams|testSbsResourceAccess)" → passed.
  • Ran: vendor/bin/phpunit tests/ClientTest.php --filter testSbsResourceAccess → passed.

Notes (optional)

  • While probing broader tests/ClientTest.php without filtering, an unrelated pre-existing test error was observed (testCustomHostname calls missing Client::setCustomHostname()); this was not introduced by the SBS changes and was left out of scope.

Update Shopee OpenAPI for Returns, Product, and SBS

Co-authored-by: nVuln <54855446+nVuln@users.noreply.github.com>
@nVuln
nVuln merged commit c6210a7 into master Aug 28, 2026
0 of 6 checks passed
@nVuln
nVuln deleted the junie/issue-49-31854268960 branch August 28, 2026 09:51
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.

Significant OpenAPI Updates of Returns, SBS and Product [Shopee-ID:1528]

2 participants