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.

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

[Maintenance] Add TwigToPdfGenerator #362

Merged
merged 1 commit into from
Apr 6, 2022

Conversation

coldic3
Copy link
Contributor

@coldic3 coldic3 commented Apr 1, 2022

Q A
Branch? master
Bug fix? no
New feature? no
Related tickets

This would give developers an easier replacement of the pdf generator if required.

@coldic3 coldic3 force-pushed the refactor/twig-to-pdf-generator branch 2 times, most recently from 8150936 to 100c9f2 Compare April 1, 2022 09:58
@coldic3 coldic3 added Maintenance Configurations, READMEs, releases, etc. DX Issues and PRs aimed at improving Developer eXperience. labels Apr 1, 2022
@coldic3 coldic3 marked this pull request as ready for review April 1, 2022 11:03
@coldic3 coldic3 requested a review from a team as a code owner April 1, 2022 11:03
@GSadee
Copy link
Member

GSadee commented Apr 4, 2022

The base of this pull-request was changed, you need fetch and reset your local branch
if you want to add new commits to this pull request. Reset before you pull, else commits
may become messed-up.

Unless you added new commits (to this branch) locally that you did not push yet,
execute git fetch origin && git reset "refactor/twig-to-pdf-generator" to update your local branch.

Feel free to ask for assistance when you get stuck 👍

@GSadee GSadee changed the base branch from 1.1 to master April 4, 2022 06:10
@GSadee GSadee force-pushed the refactor/twig-to-pdf-generator branch from 100c9f2 to 796972a Compare April 4, 2022 06:10
@coldic3 coldic3 force-pushed the refactor/twig-to-pdf-generator branch 2 times, most recently from c1da48e to 27c7019 Compare April 4, 2022 09:05
@coldic3 coldic3 force-pushed the refactor/twig-to-pdf-generator branch from 27c7019 to 19051ee Compare April 4, 2022 09:05
@coldic3 coldic3 requested a review from GSadee April 4, 2022 09:14
@GSadee GSadee merged commit bf5b2a2 into Sylius:master Apr 6, 2022
@GSadee
Copy link
Member

GSadee commented Apr 6, 2022

Thanks, Kevin! 🥇

@coldic3 coldic3 deleted the refactor/twig-to-pdf-generator branch April 6, 2022 14:08
GSadee added a commit to Sylius/InvoicingPlugin that referenced this pull request Apr 19, 2022
This PR was merged into the 1.0-dev branch.

Discussion
----------

To keep both InvoicingPlugin and RefundPlugin consistent on how PDF is generated, in this PR, I did a similar refactor to this one: Sylius/RefundPlugin#362.

Commits
-------

9598b79 [Maintenance] Add TwigToPdfGenerator
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DX Issues and PRs aimed at improving Developer eXperience. Maintenance Configurations, READMEs, releases, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants