[12.0][MIG] report_wkhtmltopdf_param

[Abranes]

Standard migration to 12.0

cc @Jortolsa @fuentes010 @ioans73 @Saphery @Willerr-mo

[gdgellatly]

@Abranes there is no way to delete a custom setting logged in as admin user or any user for that matter. I don't think it should be necessary to Become SuperUser to delete a setting. Otherwise tested and works.

[gustavovalverde]

@Abranes tested and working. Could you make the fix exposed by @gdgellatly so we can proceed with some approval love? 😄

[andres-pcg]

@Abranes nice and tidy, could you make the change suggested by @gdgellatly? it would be nice to have checks printing functionality on v12. Thanks in advance.

[andres-pcg]

Any updates on this one ?

[oca-clabot]

Hey @Abranes, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/cla
Here is a list of the users:

• Miku Laitinen miku@avoin.systems (no github login found)

Appreciation of efforts,
OCA CLAbot

[Abranes]

Hello, sorry for the late response.
@gdgellatly I added a new access rule for Administration/Settings to give permission to them to delete a setting.

I don't know what happen with @oca-clabot because I signed the CLA, anyone can review?

[gdgellatly]

@Abranes don't worry about CLA. It happens due to historic commits on migrated modules.

[andres-pcg]

Hey so this is ready @pedrobaeza , thanks

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖