Skip to content

Conversation

@jero-odoo
Copy link
Contributor

New doc outlining basic configuration for DIDWW voip provider (both in the DIDWW system and in Odoo).

@jero-odoo jero-odoo self-assigned this Sep 17, 2025
@robodoo
Copy link
Collaborator

robodoo commented Sep 17, 2025

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team September 17, 2025 19:16
@jero-odoo jero-odoo requested a review from Felicious September 17, 2025 19:34
@jero-odoo
Copy link
Contributor Author

Hey @Felicious this is ready for review.
A few things to note:
-not sure about the "Enable phone.systems" heading. I do think this is worth its own section, but I am not sure the heading is the best.
-I added the screenshot with the plus sign because I wanted to make it clear, but I am ok with it being removed

  • I was not sure about some of the parameters, so I based in on the screenshot. I do not know the why/how behind these options
    Like we discussed, I would like to do a follow up once we get this published because I would like to add some additional context if I can.
    Thanks!

Comment on lines 80 to 81
Provider` field, select *DIDWW*. Finally, enter the :guilabel:`Voip Username` and :guilabel:`Voip
Secret`, then save.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should be a guilabel? Pushing this quick fix up as a commit (:

Suggested change
Provider` field, select *DIDWW*. Finally, enter the :guilabel:`Voip Username` and :guilabel:`Voip
Secret`, then save.
Provider` field, select :guilabel:`DIDWW`. Finally, enter the :guilabel:`Voip Username` and :guilabel:`Voip
Secret`, then save.

Secret`, then save.

.. image:: didww/odoo-credentials.png
:alt: Add alt text.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
:alt: Add alt text.
:alt: DIDWW provider, username, and secret credentials entered.

@Felicious
Copy link
Contributor

@robodoo r+

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants