Skip to Content
Menu
This question has been flagged
2 Replies
2117 Views

Hi All,

I am trying out webhooks and watch a few youtube clips including Kevin Zaki's awesome vid.

I have followed along however I keep getting the same error: NameError: name 'request' is not defined. 

Does anyone know how to fix?

Thank you for reading.

Avatar
Discard

can you post your code or video that you reference?

Author

Sure here is the vid: https://youtu.be/wyU0SjOQXRk

The target record is: model.browse(int(payload.get('id'))

The code to execute is:
try:
data = request.get_json_data()
except:
data = str(request.get_http_params())

id = data['id']
message = data['message']

env['mail.message'].create({
'author_id': 2,
'model': 'sale.order',
'res_id': id,
'body': message,
'message_type': 'comment',
'subtype_id': 2,
})

The postman xml:
{
"id": 1,
"message": "Hello"
}

dunno mate. I try following the tutorial. It's working fine. No "NameError: name 'request' is not defined"

All I can suggest is try check your odoo 17 base addons/base_automation/models/base_automation.py

check if "from odoo.http import request" is on the top. if it doesn't exist, add it manually (because mine is exist). or you can download the odoo again (because it's fresh so the bug should be fixed)

someone might accidentally delete those line or you download when this is still issue (idk i'm not maintaining the odoo)

after that, try restart the server and upgrade "Automation Rules"

Best Answer

Hi,
As per the shared video, the website app has to be installed in order to get the request library.

So if its fine for you, install the website app and see.

Update:
it is not necessary to install the website app, but the code can be changed as follows:

id = payload['id']

message = payload['message']


env['mail.message'].create({

'author_id': 2,

'model': 'sale.order',

'res_id': id,

'body': message,

'message_type': 'comment',

'subtype_id': 2,

})



Thanks

Avatar
Discard
Author

Doh.... Yep, that was it and Yep, he did say that... Once again you have nailed it Niyas.

On further checking, it is not mandatory to install website app, you can get the passed data from the payload variable

Best Answer

Hi,

To resolve the issue make sure you've imported the 'request' correctly.

from odoo import http

from odoo.http import request


Make sure that your controller class inherits from http.Controller. The request object is only available within the HTTP request handling methods of this class. After making the changes to your Odoo module, restart the Odoo server to ensure the changes take effect.


Hope it helps

Avatar
Discard
Author

I appreciate the reply but, this is a webhook. Not a controller. This is executing code as part of an automation.

Related Posts Replies Views Activity
1
Dec 22
2801
1
Oct 24
1893
2
Aug 24
1027
1
Mar 24
2077
1
Feb 24
2365