Skip to content

Add PaginatedList documentation#2126

Merged
kpodemski merged 1 commit intoPrestaShop:9.xfrom
tleon:Add-cqrs-listing-doc
Mar 26, 2026
Merged

Add PaginatedList documentation#2126
kpodemski merged 1 commit intoPrestaShop:9.xfrom
tleon:Add-cqrs-listing-doc

Conversation

@tleon
Copy link
Copy Markdown
Contributor

@tleon tleon commented Mar 25, 2026

Questions Answers
Branch? 9.x
Description? Add PaginatedList Documentation and claude skill to generate more documentation on custom endpoints
Fixed ticket? ~
Sponsor company PrestaShop SA

@github-actions github-actions bot added the 9.x label Mar 25, 2026
Copy link
Copy Markdown
Contributor

@kpodemski kpodemski left a comment

Choose a reason for hiding this comment

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

To be honest, I'm not entirely sure if this skill is relevant to handling updates on just one page of the documentation, this way 🤔

@tleon tleon linked an issue Mar 26, 2026 that may be closed by this pull request
@tleon tleon force-pushed the Add-cqrs-listing-doc branch from b9bbe61 to 90b2e32 Compare March 26, 2026 13:26
@tleon
Copy link
Copy Markdown
Contributor Author

tleon commented Mar 26, 2026

This is mostly the first time I was using a skill for claude and it was mostly testing around the skill but in the end I thought what's the point of not committing it. If in the future we introduce some more custom operation well we'll be able to document them using this skill.

@tleon tleon requested a review from kpodemski March 26, 2026 13:28
Copy link
Copy Markdown
Contributor

@jolelievre jolelievre left a comment

Choose a reason for hiding this comment

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

I agree the skill is a bit overkill but it doesn't hurt either since it's here now

We're only at the beginning of using AI, so maybe some things won't be relevant but it's worth testing stuff, and we can always clean them later when we'll be more aware of what is relevant or not

@kpodemski kpodemski merged commit 8f7b9a9 into PrestaShop:9.x Mar 26, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Admin API doc : documenter listing via CQRS query

3 participants