-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[17.0][MIG] web_widget_open_tab: Migration to 17.0 #2671
[17.0][MIG] web_widget_open_tab: Migration to 17.0 #2671
Conversation
…inside other forms
Currently translated at 100.0% (6 of 6 strings) Translation: web-16.0/web-16.0-web_widget_open_tab Translate-URL: https://translation.odoo-community.org/projects/web-16-0/web-16-0-web_widget_open_tab/es/
/ocabot migration web_widget_open_tab |
I got an "Odoo Client Error" when testing on Runboat
|
@jcdrubay I think that 2 things were not in v16 either (or I couldn't make it work). I only was able to use that widget in an id field. I think it will be good improvements but maybe you should create an issue to ask for enhancement of this module. |
@@ -13,7 +13,7 @@ export class OpenTabWidget extends Component { | |||
var url = window.location.href; | |||
var searchParams = new URLSearchParams(url.split("#")[1]); | |||
searchParams.set("view_type", "form"); | |||
searchParams.set("id", this.props.value); | |||
searchParams.set("id", this.props.record.data.id); |
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.
That might be the problem with the change suggested. By doing this, this will only work with id field
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.
@moitabenfdz are you still working on this? |
b735f29
to
5ab1cf6
Compare
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.
Code + functional review LGTM
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.
functional & code review LGTM
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 9caf999. Thanks a lot for contributing to OCA. ❤️ |
No description provided.