-
-
Notifications
You must be signed in to change notification settings - Fork 728
[17.0][MIG] website_field_autocomplete: Migration to 17.0 #1098
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
base: 17.0
Are you sure you want to change the base?
[17.0][MIG] website_field_autocomplete: Migration to 17.0 #1098
Conversation
4304cc4 to
cfd3e54
Compare
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
|
@pedrobaeza please review |
|
Sorry, not using it. /ocabot migration website_field_autocomplete |
eugenios73
left a comment
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.
LGFM
Reyes4711-S73
left a comment
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.
LGTM
* Return parsed records from promise * Add `data-value-field` to allow for other value attributes * Fix appearance of demo data (container)
* Break controller getters into two methods to allow for easier inheritance manipulation * Add method in JS to return autocomplete initialization args * Add self.record_ids to represent searched RecordSet * Update request type to JSON in AJAX
* Switch model to website.menu to allow for anonymous access
* Use search_read in controller data getter, instead of custom implementation
* Bump version * Change openerp namespace to odoo * Add OCA icon * Remove jQuery require
Show Input with `displayField` value and generate hide field with `valueField` value
cfd3e54 to
38cdac4
Compare
38cdac4 to
82549a5
Compare
|
This PR has the |
|
@OCA/website-maintainers Please, can you merge this PR? |
Module migrated to version 17.0