-
Notifications
You must be signed in to change notification settings - Fork 645
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
Add WickedPdf.configure method #1090
Conversation
@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.
8754f4e
to
3b58fed
Compare
@mathieujobin Rubocop offenses have been fixed. |
@unixmonkey This is now ready for review/merging. |
@kevinnio Could you please update the |
@unixmonkey Done. |
@kevinnio Thank you so much! |
Adds the
WickedPdf.configure
method described in #1089. Also adds a deprecation warning forWickedPdf.config=
which can be silenced.