-
-
Notifications
You must be signed in to change notification settings - Fork 291
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
[10.0] Port most modules and fix bug 117 #109
Conversation
Please don't remove crm_claim_phone, as we plan to maintain crm_claim module as an OCA one (as we did with contract), because there's no other alternative for now. Indeed, helpdesk approach is better, but the one provided by Odoo is enterprise and there's no other replacement for now, so the quickest migration path from our customers that are in v8/v9 is to keep it. |
OK, no pb. But is crm_claim already in the OCA or not yet ? |
Not yet. We will port it when we start to move customers in 8.0/9.0 to 10.0 (6 months or so). |
click2dial is working now. |
For me, with my latest commit, everything is working now (except the bug #114 reported by invitu). Any feedback/test is welcomed. |
Remove crm_claim_phone, because there is no crm_claim module in v10 Start to port asterisk_click2dial (JS code still needs to be ported)
Add test for POS partner bug
3e41c56
to
6c68f3e
Compare
Runbot is red only because it misses the py-Asterisk lib... don't we have a way to define the python libs required by both Travis and runbot ? I think we have something for that, but I can't find an example. |
You can add a requirements.txt file on the repo's root. |
Mmm, it didn't work, it's all red now, even Travis which was green before the change. |
OK, Travis was red for some other reasons. I also found out that there was already some "requirements.txt" files inside the base_phone and asterisk_click2dial directories... but I don't know if it works if it's inside a module directory. |
@gurneyalex If you have a clue why runbot didn't install the "py-Asterisk" lib although I added a file "requirements.txt" at the root of the repo, please tell us. I'm trying to make runbot green, but it fails just for this missing lib. |
That's because this repository is not yet using Travis2docker. I have changed the configuration in runbot and launch the rebuild to see. You might also improve coverage on the repository. |
@pedrobaeza The runbot build was skipped. |
Uhm, that seems a general issue that they're still handling. See OCA/runbot-addons#105 |
@alexis-via should be solved now. |
…ain Garancher (Syleam)
I received some positive feedback of one of my users who uses the asterisk connector in production on odoo10... Travis is green. So let's merge :) |
This module breaks phonecall as it redefines the crm.phonecall model that is declared in crm_phonecall in crm repository. |
Port crm_phone, hr_phone, hr_recruitment_phone, event_phone to v10
Remove crm_claim_phone, because there is no crm_claim module in v10
Start to port asterisk_click2dial: JS code still needs to be ported. @sylvain-garancher I'll probably need your help on this ; you have much more experience than me in JS and a big part of the work is to adapt the JS code of asterisk_click2dial to the changes you made in the JS code of base_phone.