[16.0][MIG] base_phone: Migration to 16.0 #296
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: connector-telephony-12.0/connector-telephony-12.0-base_phone Translate-URL: https://translation.odoo-community.org/projects/connector-telephony-12-0/connector-telephony-12-0-base_phone/ [UPD] Update base_phone.pot Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: connector-telephony-12.0/connector-telephony-12.0-base_phone Translate-URL: https://translation.odoo-community.org/projects/connector-telephony-12-0/connector-telephony-12-0-base_phone/ [UPD] Update base_phone.pot Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: connector-telephony-12.0/connector-telephony-12.0-base_phone Translate-URL: https://translation.odoo-community.org/projects/connector-telephony-12-0/connector-telephony-12-0-base_phone/ [UPD] Update base_phone.pot Update translation files Updated by Update PO files to match POT (msgmerge) hook in Weblate. Update translation files Updated by Update PO files to match POT (msgmerge) hook in Weblate. [UPD] Update base_phone.pot [UPD] Update base_phone.pot [UPD] Update base_phone.pot OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex [UPD] pin flake8 version [ci skip] pre-commit update [ci skip] [UPD] .travis.yml: Use verbose colored pre-commit [skip ci]
…reformatting) from the asterisk_click2dial module to a new base_phone module.
…n phone numbers).
…rmat='xxx' arguments.
- move more asterisk-independant code to base_phone module and create new asterisk-independant modules - rename module asterisk_popup to base_phone_popup - add ability to get callerid names not only from partners, but also from leads, employees, applicants, etc... - same for auto pop-up : not only for partners, but also for leads, employees, etc...
…use it avoids the double inherit of views with phone fields (4 view XML files are removed in asterisk_click2dial* modules thanks to this !) The "Open Calling Partner" feature is now located in the top right menu, next to the Preferences (instead of the Sales menu) A click on tel: and fax: links doesn't open a new tab any more Update description and copyrights of asterisk_click2dial_crm
Convert regular string to translatable string
…letely new behavior As a consequence, the modules asterisk_click2dial_event and asterisk_click2dial_crm_claim have been dropped New wizard "Number not found" (inspired by the old "Open Calling Partner" with many enhancements), which is called by base_phone_popup and Open Caller Formatting of number for click2dial is now handled by the phonenumbers lib: as a consequence, 3 fields have been dropped from asterisk.server (national_prefix, international_prefix, country_prefix) New option 'dial_button_invisible' in the phone widget New sample script asterisk_click2dial/scripts/openerp_popup_timeout.sh Update translations and some module descriptions All the modules are now PEP8 compliant
…%' myself Add logger.error in the function used by the inherit of write/create, for easier debugging when it is called by a webservice Add . in PATH in my sample shell scripts Code cleanup (thank you flake8 !)
…he module Update module description
As a consequence, we can have non E.164 values inside the phone/mobile/fax fields: in this case, we don't display an href link on the field and we don't display the Dial button. Fix show old number on paging for fax field
…ual vs no raise with partner create/write is via code
FIX to display the result of the reformat phonenumber wizard to the user
Remove old-style icons on buttons
In order to get visibility on https://www.odoo.com/apps the OCA board has decided to add the OCA as author of all the addons maintained as part of the association.
Re-organise the codebase to prepare the arrival of a new module for a telephony system other than Asterisk
…all with XML data for res.company + it's related partner with phone numbers FIX install wizard
… from the country of the object (partner, lead, hr.applicant), not from the country of the user... a very wanted feature !
Use the new phone attributes on objects inside the code
Migrate README to new subdir format Large code clean-up Remove dead lines self.env.user.company_id => self.env.company
Currently translated at 89.2% (50 of 56 strings) Translation: connector-telephony-14.0/connector-telephony-14.0-base_phone Translate-URL: https://translation.odoo-community.org/projects/connector-telephony-14-0/connector-telephony-14-0-base_phone/es_AR/
Currently translated at 100.0% (56 of 56 strings) Translation: connector-telephony-14.0/connector-telephony-14.0-base_phone Translate-URL: https://translation.odoo-community.org/projects/connector-telephony-14-0/connector-telephony-14-0-base_phone/fr/
leemannd
approved these changes
Apr 24, 2023
|
Please anonymise the test datas |
|
|
nguyenminhchien
force-pushed
the
16.0-mig-base_phone
branch
from
460e269 to
bf414f6
Compare
@StephaneMangin : updated, please review. |
mmequignon
approved these changes
May 11, 2023
|
This PR has the |
StephaneMangin
suggested changes
May 15, 2023
base_phone/tests/test_base_phone.py
Outdated
Comment on lines
73
to
74
| "phone": "(0) 21 619 10 10", | ||
| "mobile": "(0) 79 606 42 42", |
There was a problem hiding this comment.
These numbers are camptocamp ones. Please fully anonymise all the data.
There was a problem hiding this comment.
Good day @StephaneMangin, new PR has been created to replace this PR and to update the test data.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR replaces #294