[IMP] payment_xendit: registration link #12211
Conversation
|
|
There was a problem hiding this comment.
If this is requested by Xendit, they should probably use a different heuristic. People can drop the query param, already have an account, register through a different page...
Payment providers should typically determine if an account is connected to Odoo by inspecting the payload of the payment requests.
|
hello @AntoineVDV , Janice here from the Indonesia office. I discussed with Xendit regarding this link. Unfortunately they currently don't have tracker to detect if the client is from Odoo. So, the token link will be the easiest way to track whether a client is using the Xendit integration from Odoo or not. It's not perfect because indeed like what you said: some of them may have an account already, register through a different page, etc. It's something that they are aware too, but if we put it in our documentation, it can make people who read it to register through the token and at least we have a way to track them. And, the purpose of this link is to keep track of Odoo's clients and therefore if any of the clients face issue, they can know its from Odoo and help them directly. Let me know what you think about this and feel free to tell me if you need further clarification from Xendit. Thank you! |
|
Hi @jgr-odoo, I'm not against merging this, but I think it'll be almost useless to them. Most providers use a heuristic to deterministically determine if a merchant is using Odoo, instead of relying on the registration link, and only being right sometimes. I'll let @auva-odoo review the PR because I don't know if we have guidelines against affiliate links in the documentation... I guess not 😀 |
There was a problem hiding this comment.
@AntoineVDV No, we don't have any guidelines regarding tracked links so it's fine by me. @nni-odoo I did add a suggestion to fix the formatting and revert the wording to what is was. (Also, could you please make sure not to go over the 100th-character limit while we're at it 😉 ) Thank you!
f6619a7 to
c8e4415
Compare
|
@auva-odoo thanks for the feedback! |
Update the registration link to allow Xendit to track the accounts coming from Odoo
c8e4415 to
21d13d6
Compare
Update the registration link to allow Xendit to track the accounts coming from Odoo closes #12211 Signed-off-by: Audrey Vandromme (auva) <auva@odoo.com>
Update the registration link to allow Xendit to track the accounts coming from Odoo