-
Notifications
You must be signed in to change notification settings - Fork 23.1k
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
[FW][FIX] website: fix typo in exit full screen message #165854
[FW][FIX] website: fix typo in exit full screen message #165854
Conversation
@dylankiss @rdeodoo cherrypicking of pull request #164893 failed. stdout:
stderr:
Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?). In the former case, you may want to edit this PR message as well. More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port |
With the conversion of inline templates in [1], a typo was introduced that changed "screen" to "string". Also the string was escaped and not displayed correctly. This commit fixes these issues. [1]: odoo@5700cc6 X-original-commit: aec6a8b
c13e1de
to
5061d75
Compare
@@ -25,7 +25,7 @@ export class FullscreenIndication extends Component { | |||
} | |||
|
|||
get fullScreenIndicationText() { | |||
return markup(sprintf(escape(_t("Press %(key)s to exit full screen")), "<span>esc</span>")); | |||
return markup(sprintf(escape(_t("Press %(key)s to exit full screen")), {key: "<span>esc</span>"})); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@odoo/rd-security fw-port (escaped content): #164893 (comment)
@robodoo override=ci/security |
@robodoo r+ |
With the conversion of inline templates in [1], a typo was introduced that changed "screen" to "string". Also the string was escaped and not displayed correctly. This commit fixes these issues. [1]: odoo@5700cc6 closes odoo#165854 X-original-commit: aec6a8b Signed-off-by: Romain Derie (rde) <rde@odoo.com> Signed-off-by: Dylan Kiss (dyki) <dyki@odoo.com>
With the conversion of inline templates in [1], a typo was introduced that changed "screen" to "string". This commit fixes that typo and updates the .pot file.
Also, the
escape
function used was changed to the correct one.[1] 5700cc6
Forward-Port-Of: #165442
Forward-Port-Of: #164893