Skip to content

Resource route collection factory#987

Merged
loic425 merged 3 commits into
Sylius:new-resource-loaderfrom
loic425:resource-route-collection-factory
Mar 7, 2025
Merged

Resource route collection factory#987
loic425 merged 3 commits into
Sylius:new-resource-loaderfrom
loic425:resource-route-collection-factory

Conversation

@loic425

@loic425 loic425 commented Feb 21, 2025

Copy link
Copy Markdown
Member
Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Related tickets
License MIT

@loic425 loic425 marked this pull request as draft February 21, 2025 09:05
@loic425 loic425 force-pushed the resource-route-collection-factory branch from 66c90b2 to 444010d Compare February 21, 2025 09:11
@loic425 loic425 marked this pull request as ready for review February 21, 2025 09:17
Comment thread src/Component/src/Symfony/Routing/Factory/AttributesOperationRouteFactory.php Outdated
@loic425 loic425 self-assigned this Feb 21, 2025
@loic425 loic425 force-pushed the resource-route-collection-factory branch 3 times, most recently from 0d67aae to 3603e04 Compare February 25, 2025 15:39
Comment thread tests/Application/config/routes/sylius_resource.yaml Outdated
@loic425 loic425 force-pushed the resource-route-collection-factory branch 3 times, most recently from e55b412 to 4408755 Compare March 3, 2025 08:16
Comment thread src/Bundle/Resources/config/services/routing.xml Outdated
@loic425 loic425 force-pushed the resource-route-collection-factory branch 2 times, most recently from 847d637 to 2063108 Compare March 3, 2025 10:18
@loic425 loic425 force-pushed the resource-route-collection-factory branch from 2063108 to 3bf24e4 Compare March 3, 2025 10:19
Comment thread src/Component/src/Symfony/Routing/Loader/ResourceLoader.php Outdated
Comment thread src/Component/src/Symfony/Routing/Loader/ResourceLoader.php Outdated
Comment thread src/Component/src/Symfony/Routing/Loader/ResourceLoader.php Outdated
@loic425 loic425 force-pushed the resource-route-collection-factory branch from d517c63 to ee4b84d Compare March 5, 2025 08:02
Comment thread src/Component/tests/Symfony/Routing/Loader/ResourceLoaderTest.php Outdated
@loic425 loic425 force-pushed the resource-route-collection-factory branch from ee4b84d to f8db142 Compare March 6, 2025 09:11
Co-authored-by: Dmitri Perunov <diimpp@gmail.com>
@loic425 loic425 force-pushed the resource-route-collection-factory branch from 0a9941e to 81f832e Compare March 7, 2025 07:27
Comment thread src/Bundle/Resources/config/services/routing/resource.xml Outdated
@loic425 loic425 merged commit afcad4f into Sylius:new-resource-loader Mar 7, 2025
@loic425 loic425 deleted the resource-route-collection-factory branch March 7, 2025 14:52
@loic425 loic425 mentioned this pull request Mar 7, 2025
GSadee added a commit that referenced this pull request Mar 13, 2025
| Q               | A
| --------------- | -----
| Bug fix?        | no
| New feature?    | yes
| BC breaks?      | no
| Deprecations?   | no
| Related tickets |
| License         | MIT

This work have ben reviewed via multiple PRs.
#985
#986
#987

It reworks the routing system to allow more routing system than the
attributes one in the near future.
GSadee added a commit to Sylius/Resource that referenced this pull request May 28, 2025
| Q               | A
| --------------- | -----
| Bug fix?        | no
| New feature?    | yes
| BC breaks?      | no
| Deprecations?   | no
| Related tickets |
| License         | MIT

This work have ben reviewed via multiple PRs.
Sylius/SyliusResourceBundle#985
Sylius/SyliusResourceBundle#986
Sylius/SyliusResourceBundle#987

It reworks the routing system to allow more routing system than the
attributes one in the near future.
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.

2 participants