WIP: [FIX] longpolling is replace by websocket since 16.0 version of odoo in reverse proxy #30

Open
njeudy wants to merge 2 commits from odoo_fix_webhook_url into master
njeudy commented 1 year ago

En fait j'ai essayé avec bash de faire

Odoo sort "Odoo server 16.0" comme info

 version=`/opt/odoo/custom/src/odoo/odoo-bin --version`
 version=$((${version: -4:2}))

et ensuite de comparer les versions, mais je crois que je ne suis pas au bon endroit pour le odoo-bin ..

Donc j'ai décidé en attendant d'ajouter un option "version" comme workers dans le compose.yml. Clairement on peut faire mieux :) mais j'en avais besoin.

En fait j'ai essayé avec bash de faire Odoo sort "Odoo server 16.0" comme info ``` version=`/opt/odoo/custom/src/odoo/odoo-bin --version` version=$((${version: -4:2})) ``` et ensuite de comparer les versions, mais je crois que je ne suis pas au bon endroit pour le odoo-bin .. Donc j'ai décidé en attendant d'ajouter un option "version" comme workers dans le compose.yml. Clairement on peut faire mieux :) mais j'en avais besoin.
njeudy added 1 commit 1 year ago
njeudy force-pushed odoo_fix_webhook_url from 0be34fa2b5 to c4081b2f78 1 year ago
njeudy changed title from [FIX] longpolling is replace by websocket since 16.0 version of odoo in reverse proxy to WIP: [FIX] longpolling is replace by websocket since 16.0 version of odoo in reverse proxy 1 year ago
njeudy requested review from vlab 1 year ago
Poster

@vlab ben j'y arrive pas .. mm avec juste l'option dans le compose .. je comprend pas ..

@vlab ben j'y arrive pas .. mm avec juste l'option dans le compose .. je comprend pas ..
Poster
Voila le lien de la doc odoo: https://www.odoo.com/documentation/16.0/administration/install/deploy.html?highlight=nginx#https
njeudy added 1 commit 1 year ago

Reviewers

vlab was requested for review 1 year ago
This pull request has changes conflicting with the target branch.
odoo-tecnativa/lib/common
odoo-tecnativa/metadata.yml
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.