[16.0][FIX] survey_certification_branding

[edescalona]

@BinhexTeam

When a portal user registers and completes a certification, the following errors occur:

• When the text in the "Company Name" field in the "Certification Mark" section and the survey name are too long, they overlap with the date and image values ​​for the "Certification Mark." Therefore, limiting these to 100 and 180 characters, respectively, is recommended.
• The image configured in the "Certification Mark" section of the survey is not displayed.

This change request resolves both of these issues.

Survey title:

Company name (Certification branding)

[FIX] Before (Certification Modern)

[FIX] Before (Certification Classic)

After (Certification Modern)

After (Certification Classic)

[edescalona]

ping @rrebollo

[atgalvez08]

The license badge was changed from license to licence. OCA consistently uses license in badges, and the README is autogenerated, so this change should be reverted or applied in the readme source instead.

[edescalona]

Hi @atgalvez08 , that's how the pre-commit generates it; perhaps it's a missing configuration. @pedrobaeza , could you comment on this? Thanks

[pedrobaeza]

This is because the README template that uses ocabot is not the repository one, but a custom fixed one, provoking this problem. @sbidoul is finally the template included in the latest oca-addons-repo-template?

[edescalona]

ping @atgalvez08 @rrebollo

[rrebollo]

@edescalona Good catch! Did you consider handling the text overflow with CSS (SCSS)?

[edescalona]

Hi @rrebollo , yes we looked into that case with CSS, but it's not the best approach here. The goal is to avoid very long content and the certificate stretching across two pages.

[edescalona]

Hi @sbidoul , could you please review @pedrobaeza comment or tell us how to proceed? Thanks

[pedrobaeza]

I think that if you update copier template to the latest one, now it's included.