-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
odoo: replace longpolling_port with gevent_port #16855
Conversation
longpolling_port is deprecated in doo16 see https://github.com/odoo/odoo/blob/16.0/odoo/tools/config.py#L565 Signed-off-by: Amin Cheloh <amincheloh@gmail.com> Signed-off-by: Amin Cheloh <amincheloh@gmail.com>
Hi @amincheloh, Thanks for your contribution! Although your change is rather simple, we need to perform some further investigation on our side. I have opened an internal task to review it and our team will get back to you when they have any update. I'm moving this to on-hold in the meantime. |
@gongomgra do you have any updates? |
Hi @amincheloh, I'm afraid our engineering team didn't have time to work on this yet, but they will post any update here once they do. |
why is this still not merged? This issue still remains in v17 |
Hi @1moere1, Thanks for your interest in using Bitnami. I'm afraid we didn't have time to work on this on our side yet. I will ping the engineering team about this. |
Hi @amincheloh, Thanks for your contribution and your interest in using Bitnami. Unfortunately, your change was not the only one required to fix this issue, and we have released a new image with many other generic changes required for this to work in all supported versions. You can find more information in the commit linked below We are closing this PR now. Sorry for the inconveniences. |
longpolling_port is deprecated in doo16 see https://github.com/odoo/odoo/blob/16.0/odoo/tools/config.py#L565
Signed-off-by: Amin Cheloh amincheloh@gmail.com
Description of the change
Benefits
Possible drawbacks
Applicable issues
Additional information