OCA / connector-telephony

Odoo modules for telephony integration
GNU Affero General Public License v3.0
151 stars 293 forks source link

[ADD] connector_voicent #191

Closed murtuzasaleh closed 5 years ago

murtuzasaleh commented 5 years ago

186

Voicent Connector

This module allows you to connect Odoo with Voicent https://www.voicent.com and is meant to be extended to integrate Odoo records and processes with phone calls made by Voicent.

oca-clabot commented 5 years ago

Hey @murtuzasaleh, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet. You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/cla Here is a list of the users:

Appreciation of efforts, OCA CLAbot

murtuzasaleh commented 5 years ago

@max3903 Can you please review this PR.

max3903 commented 5 years ago

@murtuzasaleh Looks good. I will run some tests next week.

murtuzasaleh commented 5 years ago

@max3903
1

1) I don't think we need this condition because it is worked based on next call so when the job is created pass the Next Call value in eta so the job is to execute based on that.

2

2) When the new job is created how we can pass the date in eta field for the execution of this code because we can not pass Next Call field value on it, and job is execution part does not depend on the Scheduler, we can delay it based on eta and pass it where the method is defined.

Correct me if I am wrong.

max3903 commented 5 years ago

@murtuzasaleh

  1. Correct.
  2. The job is created with the link to the backend (where the next call value is stored) so the job knows at what time it should be executed.