Skip to content
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

[16.0][mig] web_widget_url_advanced: Migration to 16.0 #2431

Open
wants to merge 28 commits into
base: 16.0
Choose a base branch
from

Conversation

dsolanki-initos
Copy link
Contributor

No description provided.

simahawk and others added 28 commits March 7, 2023 15:14
On tree views the widget will be initialized once.
Without this change the value displayed is always the same.
Now anchor's text is computed on rendering.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: web-14.0/web-14.0-web_widget_url_advanced
Translate-URL: https://translation.odoo-community.org/projects/web-14-0/web-14-0-web_widget_url_advanced/
@dsolanki-initos dsolanki-initos mentioned this pull request Mar 7, 2023
45 tasks
Copy link
Contributor

@simahawk simahawk left a comment

Choose a reason for hiding this comment

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

Please rewrite the migration commit (= to the PR title sounds good)

*/
_renderReadonly: function () {
// Base widget uses `this.attrs.text` instead of `this.value` when available.
// TODO: To check better way for update link
Copy link
Contributor

Choose a reason for hiding this comment

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

can you check this TODO pls?

@simahawk
Copy link
Contributor

simahawk commented Mar 9, 2023

/ocabot migration web_widget_url_advanced

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Mar 9, 2023
@simahawk
Copy link
Contributor

simahawk commented Mar 9, 2023

BTW any chance to propose this features to odoo master?

@Vang-NguyenPhu
Copy link

Vang-NguyenPhu commented Dec 13, 2023

The feature doesn't work for me, could you please review again sir?

Here is the code:
image

And here is the result:
image

Please correct me if I missed anything, thank you.

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.