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

[MIG] web_edit_user_filter: Migration to 15.0 #2746

Open
wants to merge 23 commits into
base: 15.0
Choose a base branch
from

Conversation

amh-mw
Copy link

@amh-mw amh-mw commented Feb 13, 2024

No description provided.

@amh-mw amh-mw force-pushed the 15.0-mig-web_edit_user_filter branch 4 times, most recently from 48e7507 to 33cd2b8 Compare March 20, 2024 15:48
@amh-mw amh-mw force-pushed the 15.0-mig-web_edit_user_filter branch 2 times, most recently from 1a2260d to d2de92a Compare June 12, 2024 16:13
tarteo and others added 22 commits July 19, 2024 15:22
[FIX] Readme

[FIX] Lint

[ADD] Tests

[IMP] UI/UX

[IMP] Hide popover when other is opened

[IMP] Add item to roadmap
Added translation using Weblate (Chinese (Simplified))

Translated using Weblate (Chinese (Simplified))

Currently translated at 100.0% (3 of 3 strings)

Translation: web-12.0/web-12.0-web_edit_user_filter
Translate-URL: https://translation.odoo-community.org/projects/web-12-0/web-12-0-web_edit_user_filter/zh_CN/

Added translation using Weblate (Croatian)

Translated using Weblate (Croatian)

Currently translated at 66.7% (2 of 3 strings)

Translation: web-12.0/web-12.0-web_edit_user_filter
Translate-URL: https://translation.odoo-community.org/projects/web-12-0/web-12-0-web_edit_user_filter/hr/

Added translation using Weblate (Portuguese)

Translated using Weblate (Portuguese)

Currently translated at 100.0% (3 of 3 strings)

Translation: web-12.0/web-12.0-web_edit_user_filter
Translate-URL: https://translation.odoo-community.org/projects/web-12-0/web-12-0-web_edit_user_filter/pt/
has to explicit check for type of value !== null as the typeof null is object
https://i.stack.imgur.com/FzI1R.png
Added translation using Weblate (Danish)

Translated using Weblate (Danish)

Currently translated at 100.0% (3 of 3 strings)

Translation: web-12.0/web-12.0-web_edit_user_filter
Translate-URL: https://translation.odoo-community.org/projects/web-12-0/web-12-0-web_edit_user_filter/da/
Added translation using Weblate (Spanish)

Translated using Weblate (Spanish)

Currently translated at 100.0% (3 of 3 strings)

Translation: web-12.0/web-12.0-web_edit_user_filter
Translate-URL: https://translation.odoo-community.org/projects/web-12-0/web-12-0-web_edit_user_filter/es/

Added translation using Weblate (French)

Translated using Weblate (French)

Currently translated at 100.0% (3 of 3 strings)

Translation: web-12.0/web-12.0-web_edit_user_filter
Translate-URL: https://translation.odoo-community.org/projects/web-12-0/web-12-0-web_edit_user_filter/fr/

Added translation using Weblate (Catalan)

Translated using Weblate (Catalan)

Currently translated at 100.0% (3 of 3 strings)

Translation: web-12.0/web-12.0-web_edit_user_filter
Translate-URL: https://translation.odoo-community.org/projects/web-12-0/web-12-0-web_edit_user_filter/ca/
Added translation using Weblate (Portuguese (Brazil))

Translated using Weblate (Portuguese (Brazil))

Currently translated at 100.0% (3 of 3 strings)

Translation: web-12.0/web-12.0-web_edit_user_filter
Translate-URL: https://translation.odoo-community.org/projects/web-12-0/web-12-0-web_edit_user_filter/pt_BR/
When clicking to edit a filter in search view, the _process_filters
function will receive the faceID argument as a string, which will
be compared to facet.groupId that is an Integer.

Therefore we should use == operator when comparing this value to
allow type coercion from JS.
@amh-mw amh-mw force-pushed the 15.0-mig-web_edit_user_filter branch from d2de92a to 112f751 Compare July 22, 2024 15:44
@amh-mw amh-mw marked this pull request as ready for review July 22, 2024 15:47
@amh-mw
Copy link
Author

amh-mw commented Jul 22, 2024

Unable to reproduce pre-commit failure locally on macOS Sonoma 14.5 with either pre-commit 2.20.0 or 3.7.1
I did a pre-comment uninstall then pre-commit install and that allowed me to reproduce.

@amh-mw amh-mw force-pushed the 15.0-mig-web_edit_user_filter branch from 112f751 to 871502c Compare July 22, 2024 18:59
@amh-mw amh-mw force-pushed the 15.0-mig-web_edit_user_filter branch from 871502c to 328b109 Compare July 29, 2024 16:55
@amh-mw amh-mw mentioned this pull request Sep 3, 2024
47 tasks
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.