Skip to content

Add manufacturer detail endpoint#113

Draft
MattKelvin wants to merge 18 commits intoPrestaShop:devfrom
MattKelvin:feature/add-manufacturer-detail-endpoint
Draft

Add manufacturer detail endpoint#113
MattKelvin wants to merge 18 commits intoPrestaShop:devfrom
MattKelvin:feature/add-manufacturer-detail-endpoint

Conversation

@MattKelvin
Copy link

@MattKelvin MattKelvin commented Nov 22, 2025

Questions Answers
Description? See below
Type? new feature
BC breaks? no
Deprecations? no
Fixed ticket? -
Sponsor company Mademoiselle bio
How to test? See below

Add manufacturer detail endpoint. Needs #63 to be merged. Need a fix for PrestaShop/PrestaShop#39469.

New endpoints

GET /manufacturer/1/details/1 (manufacturer id and language id)
  1. Setup
  • Install Prestashop 9.0.x.
  • Install the ps_apiresources module from this branch.
  • Navigate to modules/ps_apiresources and run:
    composer install
  • Clear the cache
    composer clear-test-cache
  1. API Configuration
  • In the API Admin, create a new client with the following permissions:
    -- manufacturer_read
    -- manufacturer_write
  • Disable Security Mode in the API Admin.
  1. cURL Examples
  • In the examples below, replace:
    -- YOUR_CLIENT_ID
    -- YOUR_CLIENT_SECRET
    -- YOUR_DOMAIN_NAME
    -- YOUR_ACCESS_TOKEN

How to test :

  1. Get a new access token :
curl --location 'YOUR_DOMAIN_NAME/admin-api/access_token' \
--form 'grant_type="client_credentials"' \
--form 'client_id="YOUR_CLIENT_ID"' \
--form 'client_secret="YOUR_CLIENT_SECRET"' \
--form 'scope[]="manufacturer_read"' 
  1. Get a manufacturer detail

⚠️ test are ok but in production we have a warning which break it. See: PrestaShop/PrestaShop#39470

curl --location 'YOUR_DOMAIN_NAME/admin-api/manufacturer/1/details/1' --header 'Authorization: Bearer YOUR_ACCESS_TOKEN'

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

Projects

Status: Ready for review

Development

Successfully merging this pull request may close these issues.

2 participants