-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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.
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
[WIP][Documentation] Customizing credit memo model itself #12492
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am wondering about the last example, shouldn't we describe the overriding credit memo by adding some extra field to this entity instead of rendering extra data from the different controller? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, however, I do agree with @GSadee's comment.
Custom background colors | ||
------------------------ | ||
|
||
The first exemplary customization is to change background color of the heading of line items table. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shall we add some screenshots here?
There's an issue with overriding an entity - it does not work currently: Sylius/RefundPlugin#278 |
7b0ee4b
to
5521bd1
Compare
Anyway, I've prepared docs in case the linked issue is fixed - the last commit includes them. If not, we can still go with the first version, which is included in the first commit. |
This PR was merged into the 1.9 branch. Discussion ---------- | Q | A | --------------- | ----- | Branch? | master | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Related tickets | part one of @pamil's pr #12492 | License | MIT <!-- - Bug fixes must be submitted against the 1.7 or 1.8 branch (the lowest possible) - Features and deprecations must be submitted against the master branch - Make sure that the correct base branch is set To be sure you are not breaking any Backward Compatibilities, check the documentation: https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html --> Commits ------- 7d55aca [Documentation] Customizing credit memo
5521bd1
to
c09737f
Compare
c09737f
to
421504a
Compare
The base of this pull-request was changed, you need fetch and reset your local branch Unless you added new commits (to this branch) locally that you did not push yet, Feel free to ask for assistance when you get stuck 👍 |
Basic customization of RefundPlugin templates.