Skip to content

Add a bridge for a Joli Media Sylius Admin#96

Draft
loic425 wants to merge 101 commits intojolicode:mainfrom
loic425:joli-sylius-admin
Draft

Add a bridge for a Joli Media Sylius Admin#96
loic425 wants to merge 101 commits intojolicode:mainfrom
loic425:joli-sylius-admin

Conversation

@loic425
Copy link
Copy Markdown
Contributor

@loic425 loic425 commented Mar 24, 2026

sylius-ecommerce-media-explore sylius-media-details
  • Handle pagination sizes from the config
  • Uniformize service ids
  • Remove demo config reference file
  • Use existing translations on grid headers
  • Create directory button on main actions
  • Remame directory on grid
  • Delete directory on grid
  • Improve media show breadcrumbs
  • Remove tree
  • Add directories on grid view
  • Improve post type to use the media choice type
  • Do not have show the tabs on media details when media has no variations
  • Use the post picture on the post listing
  • Fix the variation regenerate link (currently connected with the EasyAdmin brige)
  • Supports both Sylius Stack & Sylius E-commerce
  • Add missing actions on the media listing
    • Rename a media
    • Delete a media
  • Fix icon attributes size on _as_image.html.twig
  • Handle no media results
  • Do not show pagination when not needed
  • Add paginator on the grid view
  • Improve copy to clipboard UX
  • Add a live collection for post medias on the demo
  • Add missing actions on the media details page
    • Rename a media
    • Delete a media
    • Move a media to another directory
  • Add missing actions on the choose modal
    • create a new directory
  • Add the bridge on the test application
  • PHPUnit tests
    • explore
    • media details
    • delete media
    • delete directory
    • upload media
    • move media
    • create directory
    • choose media
    • rename media
    • rename directory
  • Fix the templates on Sylius admin bundle (Fix admin templates Sylius/Sylius#18972)
  • Fix the NullMedia issue
  • Fix QA
  • Fix breadcrumb refresh on move media modal
  • Add a breadcrumb in the choose media modal
  • Improve pagintation on choose media modal

@loic425 loic425 marked this pull request as draft March 24, 2026 09:03
@loic425 loic425 changed the title Add bridge for a Joli Sylius Admin Add a bridge for a Joli Sylius Admin Mar 24, 2026
@loic425 loic425 changed the title Add a bridge for a Joli Sylius Admin Add a bridge for a Joli Media Sylius Admin Mar 24, 2026
@loic425 loic425 force-pushed the joli-sylius-admin branch from 3bcbb01 to cac316f Compare March 25, 2026 09:09
@@ -0,0 +1,20 @@
# Full configuration reference: https://symfony.com/bundles/ux-icons/current/index.html#configuration
ux_icons:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Question innocente, c'est une dépendance de Sylius, ou de notre bridge ?

Copy link
Copy Markdown
Contributor Author

@loic425 loic425 Mar 26, 2026

Choose a reason for hiding this comment

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

Le sylius/bootstrap-admin-ui tire cette dépendance et elle est géniale 👯

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.

FYI, the admin bundle from Sylius E-Commerce also uses the Symfony UX icon packages.

@loic425 loic425 force-pushed the joli-sylius-admin branch 6 times, most recently from 88b22db to ea10bb0 Compare April 3, 2026 09:28
@loic425 loic425 force-pushed the joli-sylius-admin branch from f678ab3 to 4157579 Compare April 9, 2026 12:08
@loic425 loic425 force-pushed the joli-sylius-admin branch from cf47f53 to ac4c74e Compare April 9, 2026 16:09
@loic425 loic425 force-pushed the joli-sylius-admin branch from ac4c74e to 677780c Compare April 10, 2026 09:27
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.

3 participants