Skip to content

REST API constructor parameter validation#414

Merged
keharper merged 6 commits into
mainfrom
kh_webapi
Oct 6, 2025
Merged

REST API constructor parameter validation#414
keharper merged 6 commits into
mainfrom
kh_webapi

Conversation

@keharper

@keharper keharper commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

Purpose of this pull request

This pull request (PR) adds info about valid and invalid types of REST API constructor parameters.

Security bulletin
KB topic

Affected pages

  • ...

Links to Magento Open Source code

  • ...

@keharper keharper self-assigned this Oct 3, 2025
@keharper keharper added major-update Significant original updates to existing content internal Differentiates work between external and internal contributors labels Oct 3, 2025
Comment thread src/pages/development/components/web-api/services.md
Comment thread src/pages/development/components/web-api/services.md Outdated
@keharper keharper merged commit 6668bd5 into main Oct 6, 2025
6 checks passed
@keharper keharper deleted the kh_webapi branch October 6, 2025 16:23
Supported parameter types:

- Simple types (string, int, float, boolean)
- Arrays of simple types

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@keharper I misunderstood about supported types in general. For constructor we are not allow type array. Could you please revert this line? Sorry about that.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixing in #415

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

Labels

internal Differentiates work between external and internal contributors major-update Significant original updates to existing content

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants