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

Bump wicked_pdf from 1.4.0 to 2.1.0 #5738

Merged
merged 1 commit into from Jul 9, 2020

Conversation

dependabot-preview[bot]
Copy link
Contributor

Bumps wicked_pdf from 1.4.0 to 2.1.0.

Changelog

Sourced from wicked_pdf's changelog.

[2.1.0] - 2020-06-14

Fixes

New Features

[2.0.2] - 2020-03-17

Fixes

[2.0.1] - 2020-02-22

Fixes

[2.0.0] - 2020-02-22

Breaking changes

  • Remove support for older Ruby and Rails versions - This project no longer supports Ruby < 2.2 and Rails < 4. It may work for you, but we are no longer worrying about breaking backwards compatibility for versions older than these. If you are on an affected version, you can continue to use the 1.x releases. Patches to fix broken behavior on old versions may not be accepted unless they are highly decoupled from the rest of the code base.

New Features

Fixes

Commits
  • 309ef96 Update VERSION to 2.1.0
  • aefa10f Add option use_xvfb to emulate an X server (#909)
  • 23c13d7 Add a option to disable/enable local file access for wkhtmltopdf 0.12.6
  • 3120646 Merge pull request #915 from weber-s/master
  • 914a310 add 'locals' option to the README advanced section
  • 3a3ccc0 Merge pull request #905 from reinaris/patch-1
  • 8e6f708 Add no_stop_slow_scripts to readme
  • 0139342 Merge pull request #896 from skillstream/master
  • 1efdba2 Support webpacker assets
  • b019317 Bump version for 2.0.2 release
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language
  • @dependabot badge me will comment on this PR with code to add a "Dependabot enabled" badge to your readme

Additionally, you can set the following in your Dependabot dashboard:

  • Update frequency (including time of day and day of week)
  • Pull request limits (per update run and/or open at any time)
  • Automerge options (never/patch/minor, and dev/runtime dependencies)
  • Out-of-range updates (receive only lockfile updates, if desired)
  • Security updates (receive only security updates, if desired)

Bumps [wicked_pdf](https://github.com/mileszs/wicked_pdf) from 1.4.0 to 2.1.0.
- [Release notes](https://github.com/mileszs/wicked_pdf/releases)
- [Changelog](https://github.com/mileszs/wicked_pdf/blob/master/CHANGELOG.md)
- [Commits](mileszs/wicked_pdf@1.4.0...2.1.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Copy link
Contributor

@luisramos0 luisramos0 left a comment

Choose a reason for hiding this comment

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

The build looks good (just a flaky spec). I wonder if there's something we need to update in ofn-install.

@Matt-Yorkley
Copy link
Contributor

Matt-Yorkley commented Jul 7, 2020

I think we should do some sanity checks for successful PDF creation. Moving to test ready...

@filipefurtad0 filipefurtad0 self-assigned this Jul 9, 2020
@filipefurtad0 filipefurtad0 added the pr-staged-uk staging.openfoodnetwork.org.uk label Jul 9, 2020
@filipefurtad0
Copy link
Contributor

Hi @luisramos0 and @Matt-Yorkley ,

Tested pdf rendering in:

  • admin/orders
  • admin/Rxxxxxxxx/edit

All works as before, this is ready to go.

As a side note, and unrelated to this PR: error 422 is still observed in the console, for some orders when bulk-printing. This does not break the functionality (the pdf is rendered) and seems to appear mostly when printing orders which have a "due balance" status, but not always. The error does not occur if the pdf is printed via admin/orders/Rxxxxxxx/print.

image

@filipefurtad0 filipefurtad0 removed the pr-staged-uk staging.openfoodnetwork.org.uk label Jul 9, 2020
@luisramos0 luisramos0 merged commit ed91500 into master Jul 9, 2020
@luisramos0 luisramos0 deleted the dependabot/bundler/wicked_pdf-2.1.0 branch July 9, 2020 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants