Skip to content

fix(Sylius): Fix media get path#121

Open
loic425 wants to merge 1 commit into
jolicode:mainfrom
loic425:fix/media-path
Open

fix(Sylius): Fix media get path#121
loic425 wants to merge 1 commit into
jolicode:mainfrom
loic425:fix/media-path

Conversation

@loic425
Copy link
Copy Markdown
Contributor

@loic425 loic425 commented May 15, 2026

I've challenged with ChatGPT but this solution seems to be the most pragmatic one.

Variante élégante

Tu pourrais aussi :

ne plus persister path à terme,
mais garder un getter virtuel via serializer/API.

Par contre ça implique :

custom hydration,
ou remplacement complet des entités.

Donc probablement overkill pour le gain réel.

Ton approche actuelle est franchement la plus pragmatique.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant