-
Notifications
You must be signed in to change notification settings - Fork 10.6k
[FIX] point_of_sale: use the correct terms in Chrome interface #2180
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
[FIX] point_of_sale: use the correct terms in Chrome interface #2180
Conversation
|
@qle-odoo |
qle-odoo
left a comment
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.
@robodoo r+
|
I'm sorry, @qle-odoo. I'm afraid I can't do that. |
|
@lse-odoo sorry i can't merge documentation |
Feyensv
left a comment
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.
@robodoo r+
(are you sure this change is not needed in the 13.0 documentation ?)
|
@robodoo squash |
|
Merge method set to squash |
No ePos in 13.0 so no doc... |
|
@robodoo r- |
If you're speaking of the commit message & title, with the 'squash' method, the merge commit will use the PR title & message ;). |
LoredanaLrpz
left a comment
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.
hello @lse-odoo , sorry I approved but didn't check the characters, I've made suggestions then I'll approve that PR :D
Because of a recent Chrome security update, a lot of customers using peripheral devices (printers, ...) will have to follow this doc. Therefore, I followed the doc and did the flow on my side. I modified 'Copy to file' to 'Export' and 'Trusted Root Certification Authorities' to 'Authorities' in order to have the exact same terms as in Chrome (I guess they updated their terms at some point).
227212e to
f65d891
Compare
|
@LoredanaLrpz Done! Is there something else that you would like me to change ? |
LoredanaLrpz
left a comment
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.
@lse-odoo looks good to me :) ! Thank you =D
|
@robodoo r+ |
Because of a recent Chrome security update, a lot of customers using peripheral devices (printers, ...) will have to follow this doc. Therefore, I followed the doc and did the flow on my side. I modified 'Copy to file' to 'Export' and 'Trusted Root Certification Authorities' to 'Authorities' in order to have the exact same terms as in Chrome (I guess they updated their terms at some point). closes #2180 Signed-off-by: Victor Feyens (vfe) <vfe@odoo.com>
Because of a recent Chrome security update, a lot of customers using peripheral devices (printers, ...) will have to follow this doc.
Therefore, I followed the doc and did the flow on my side.
I modified 'Copy to file' to 'Export' and 'Trusted Root Certification Authorities' to 'Authorities' in order to have the exact same terms as in Chrome (I guess they updated their terms at some point).