Skip to content

Conversation

@Progi1984
Copy link
Member

@Progi1984 Progi1984 commented Oct 15, 2025

Questions Answers
Branch? dev
Description? Added endpoints for domain "Store"
Type? improvement
Category? TE
BC breaks? no
Deprecations? no
How to test? ⬇️
Fixed issue or discussion? N/A
Related PRs N/A
Sponsor company lefevre.dev

How to test ?

  • Create 3 stores (with status enabled) in backoffice and keep their IDs {idStore1} {idStore2} {idStore3}
  • Create an API Client with these scopes : store_read & store_write
  • Request an access token
  • Query 1 :
    • Method : GET
    • URI : /store/{idStore1}
    • Response :
      • HTTP Code : 200
      • HTTP Body : Contains a JSON for the Store (its id & its status)
  • Query 2 :
    • Method : PUT
    • URI : /store/{idStore1}/toggle-status
    • Response :
      • HTTP Code : 204
  • Check in the BO : the Store idStore1 is disabled
  • Query 3 :
    • Method : DELETE
    • URI : /store/{idStore1}
    • Response :
      • HTTP Code : 204
  • Check in the BO : the Store idStore1 is not found (it is deleted)
  • Query 4 :
    • Method : PUT
    • URI : /stores/set-status
    • Body :
    {"storeIds": [{idStore2}, {idStore3}], "enabled": false}
    • Response :
      • HTTP Code : 204
  • Check in the BO : stores idStore2 & idStore3 are disabled
  • Query 5 :
    • Method : PUT
    • URI : /stores/delete
    • Body :
    {"storeIds": [{idStore2}, {idStore3}]}
    • Response :
      • HTTP Code : 204
  • Check in the BO : stores idStore2 & idStore3 are not found (they are deleted)

tleon
tleon previously requested changes Oct 21, 2025
@ps-jarvis ps-jarvis added the Waiting for QA Status: Waiting for QA feedback label Oct 22, 2025
@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Oct 22, 2025
@kpodemski kpodemski dismissed tleon’s stale review October 22, 2025 09:26

All the requested changes have been implemented.

@SiraDIOP SiraDIOP self-assigned this Oct 28, 2025
@SiraDIOP
Copy link

Hello @Progi1984,

Thanks for your PR, i tested it and it's good for me :)
Query 1:
Capture d’écran 2025-10-28 à 10 42 46

Query 2:
Capture d’écran 2025-10-28 à 10 46 27
BO:
Capture d’écran 2025-10-28 à 10 47 11

Query 3:
Capture d’écran 2025-10-28 à 10 50 18
BO:
Capture d’écran 2025-10-28 à 10 50 46

Query 4:

Capture d’écran 2025-10-28 à 10 53 08 BO: Capture d’écran 2025-10-28 à 10 53 42

Query 5:
Capture d’écran 2025-10-28 à 10 55 07
BO:
Capture d’écran 2025-10-28 à 10 55 43

Thanks

@SiraDIOP SiraDIOP added QA ✔️ Status: QA-Approved and removed Waiting for QA Status: Waiting for QA feedback labels Oct 28, 2025
@nicosomb nicosomb added this to the 0.2.0 milestone Oct 28, 2025
@nicosomb
Copy link
Contributor

thank you @Progi1984 and @SiraDIOP !

@nicosomb nicosomb merged commit 4305fb1 into PrestaShop:dev Oct 28, 2025
13 checks passed
@github-project-automation github-project-automation bot moved this from To be tested to Merged in PR Dashboard Oct 28, 2025
@ps-jarvis ps-jarvis moved this from Merged to To be tested in PR Dashboard Oct 28, 2025
@Progi1984 Progi1984 deleted the store branch October 28, 2025 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

7 participants