Skip to content

Conversation

@qle-odoo
Copy link
Contributor

The default protocol int the POS with "other device"
is on HTTP.

This doc explain how to activate the HTTPS in the POS

@robodoo
Copy link
Collaborator

robodoo commented Oct 18, 2021

@qle-odoo
Copy link
Contributor Author

@odoo/doc-review this one is ready now

@AntoineVDV AntoineVDV self-requested a review October 19, 2021 10:52
Copy link
Collaborator

@AntoineVDV AntoineVDV left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couldn't help but notice the typo in the commit message body 🙈

@qle-odoo
Copy link
Contributor Author

@AntoineVDV
Small changes are made except the 2 points above which are essential for the good understanding of the customers

Copy link
Contributor

@jcs-odoo jcs-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello,

Thank you for this documentation. We reviewed it together with Adrien (aim) who is going to take care of the PoS doc.

I suggested plenty of changes, I hope you don't mind.

Imo, the images aren't useful here, but if you add images, please follow the guidelines that Antoine linked in a previous comment (compress with pngquant, crop more, and put them in a separate folder)

Let me know if you need anything.

Cheers,
Jonathan

The default protocol int the POS with "other device"
is on HTTP.

This doc explain how to activate the HTTPS in the POS
Copy link
Contributor

@jcs-odoo jcs-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AntoineVDV
I just had a chat with Quentin and said I'd push my modifications so that we can merge this faster.

@jcs-odoo jcs-odoo added this to the 14.0 milestone Oct 20, 2021
@jcs-odoo jcs-odoo requested a review from AntoineVDV October 20, 2021 21:48
@pimodoo
Copy link

pimodoo commented Oct 21, 2021

@AntoineVDV Hello, could you merge this PR please?

Copy link
Collaborator

@AntoineVDV AntoineVDV left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

robodoo pushed a commit that referenced this pull request Oct 21, 2021
The default protocol int the POS with "other device"
is on HTTP.

This doc explain how to activate the HTTPS in the POS

closes #1195

Signed-off-by: Antoine Vandevenne (anv) <anv@odoo.com>
@robodoo robodoo closed this Oct 21, 2021
@robodoo robodoo temporarily deployed to merge October 21, 2021 10:40 Inactive
@fw-bot fw-bot deleted the 14.0-pos-https-qle branch November 4, 2021 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants