Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove partials, add twig components #458

Merged
merged 4 commits into from
Dec 9, 2024
Merged

Remove partials, add twig components #458

merged 4 commits into from
Dec 9, 2024

Conversation

mpysiak
Copy link
Member

@mpysiak mpysiak commented Dec 6, 2024

Q A
Branch? 2.0
Bug fix? no
New feature? no
Related tickets n/a

@mpysiak mpysiak force-pushed the SYL-4168-add-twig-components branch from 872d60d to 72a286b Compare December 6, 2024 12:04
@mpysiak mpysiak force-pushed the SYL-4168-add-twig-components branch from 72a286b to e1bc9df Compare December 9, 2024 05:46
@mpysiak mpysiak mentioned this pull request Dec 9, 2024
Merged
@GSadee GSadee added the Enhancement Minor issues and PRs improving the current solutions (optimizations, typo fixes, etc.). label Dec 9, 2024
GSadee
GSadee reviewed Dec 9, 2024
View reviewed changes
config/app/twig_hooks/shop/account/order/show.yaml Outdated Show resolved Hide resolved
config/services/twig_components.xml Outdated Show resolved Hide resolved
config/services/twig_components.xml Outdated Show resolved Hide resolved
src/Doctrine/ORM/CreditMemoRepository.php Outdated Show resolved Hide resolved
src/Repository/CreditMemoRepositoryInterface.php Outdated Show resolved Hide resolved
@GSadee GSadee merged commit df8a5a0 into Sylius:2.0 Dec 9, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GSadee GSadee GSadee approved these changes

Assignees
No one assigned
Labels
Enhancement Minor issues and PRs improving the current solutions (optimizations, typo fixes, etc.).
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mpysiak @GSadee