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

Add WickedPdf.configure method #1090

Merged
merged 4 commits into from
Feb 10, 2024

Conversation

kevinnio
Copy link
Contributor

@kevinnio kevinnio commented Jan 5, 2024

Adds the WickedPdf.configure method described in #1089. Also adds a deprecation warning for WickedPdf.config= which can be silenced.

@mathieujobin
Copy link
Contributor

@kevinnio Please look into the rubocop/rspec failures please

This methods uses the config block pattern to allow users to add config
values without risking overwriting WickedPdf.config.
This adds a deprecation warning for the old way to set config. Users
have the option to silence the warning if they want to do so.
@kevinnio kevinnio force-pushed the kevinnio/add-config-block branch from 8754f4e to 3b58fed Compare January 23, 2024 16:29
@kevinnio
Copy link
Contributor Author

@mathieujobin Rubocop offenses have been fixed.

@kevinnio kevinnio requested a review from mathieujobin January 27, 2024 22:28
@kevinnio
Copy link
Contributor Author

kevinnio commented Feb 7, 2024

@unixmonkey This is now ready for review/merging.

@unixmonkey
Copy link
Collaborator

@kevinnio Could you please update the generators/wicked_pdf/templates/wicked_pdf.rb file to use the new configuration syntax? This gets copied over if you run bin/rails generate wicked_pdf, which is part of the install instructions.

@kevinnio
Copy link
Contributor Author

kevinnio commented Feb 9, 2024

@unixmonkey Done.

@unixmonkey unixmonkey merged commit 0bc0f50 into mileszs:master Feb 10, 2024
4 of 12 checks passed
@unixmonkey
Copy link
Collaborator

@kevinnio Thank you so much!

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.

3 participants