Skip to content

improve dto documentation - #2231

Merged
soyuka merged 1 commit into
api-platform:4.2from
soyuka:dto
Dec 4, 2025
Merged

improve dto documentation#2231
soyuka merged 1 commit into
api-platform:4.2from
soyuka:dto

Conversation

@soyuka

@soyuka soyuka commented Nov 25, 2025

Copy link
Copy Markdown
Member

No description provided.

@soyuka
soyuka force-pushed the dto branch 2 times, most recently from a01ffbe to 0cc1e94 Compare November 25, 2025 15:03

@JonathanBaudoin JonathanBaudoin left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Really better, good job. 👍

Comment thread core/dto.md Outdated
Comment thread core/dto.md
Comment thread core/dto.md Outdated
Comment on lines +244 to +245
use App\Dto\DiscountBook;
use App\State\DiscountBookProcessor;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
use App\Dto\DiscountBook;
use App\State\DiscountBookProcessor;
use App\Dto\DiscountBook;
use App\Entity\Book as BookEntity;
use App\State\DiscountBookProcessor;
Comment thread core/dto.md Outdated
@soyuka
soyuka merged commit 323e977 into api-platform:4.2 Dec 4, 2025
1 check failed
@soyuka
soyuka deleted the dto branch December 4, 2025 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants