documentation icon indicating copy to clipboard operation
documentation copied to clipboard

[IMP] clarify "to" field for test message recipients, remove future tense

Open huisit opened this issue 1 month ago • 4 comments

documentation task card: https://www.odoo.com/odoo/action-4043/5257364

key changes @ line 181 / 192:

  • clarified restrictions on test phone number provided during whatsapp config (max. 5 recipients, unlimited messages)
  • added instructions for adding message recipients, link to meta documentation

other:

  • removed callout and appended first sentence to preceding paragraph
  • also removed instances of future tense ("will") to match current style; created follow-up task for full article refresh (https://www.odoo.com/odoo/action-4043/5356551)

This 17.0 PR can be FWP up to master. In the event of a merge conflict, tense changes can be left out for future work.

huisit avatar Nov 25 '25 18:11 huisit

Pull request status dashboard

robodoo avatar Nov 25 '25 18:11 robodoo

Hi @larm-odoo! This PR is ready for review

huisit avatar Nov 25 '25 18:11 huisit

@larm-odoo i've implemented your feedback! can you re-review please?

huisit avatar Dec 02 '25 19:12 huisit

One more note - this PR title should change to a [IMP] not [ADD] because Add is only used when we are adding a doc that did not exist before, anything we improve or change for existing docs is an [IMP], regardless of how little or how much we change. Unless it's a tiny fix, like ONE typo- that would be a [FIX] PR.

larm-odoo avatar Dec 03 '25 18:12 larm-odoo

@Felicious

huisit avatar Dec 18 '25 23:12 huisit

@Felicious this is ready for review, PR and commit have been updated from ADD -> IMP

huisit avatar Dec 19 '25 01:12 huisit

@robodoo fw=no

Felicious avatar Dec 19 '25 22:12 Felicious

Disabled forward-porting.

robodoo avatar Dec 19 '25 22:12 robodoo

@robodoo r+

huisit avatar Dec 20 '25 00:12 huisit