Skip to content

Conversation

@sjai-odoo
Copy link
Contributor

This PR add the documentation needed to use razorpay payment method in Point Of Sale.

task- 3762059

@robodoo
Copy link
Collaborator

robodoo commented Apr 1, 2024

@sjai-odoo sjai-odoo force-pushed the master-pos-razorpay-documentation-sjai branch from e481379 to ffdb5e4 Compare April 8, 2024 11:31
Copy link

@paso-odoo paso-odoo left a comment

Choose a reason for hiding this comment

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

@sjai-odoo
Thanks for working on this documentation.
Here are some comments that need to be taken into account.

Cheers:)

@sjai-odoo sjai-odoo force-pushed the master-pos-razorpay-documentation-sjai branch from ffdb5e4 to 575885c Compare April 9, 2024 05:00
@sjai-odoo
Copy link
Contributor Author

sjai-odoo commented Apr 9, 2024

@paso-odoo
All the suggested changes are done
Thank you

@sjai-odoo sjai-odoo force-pushed the master-pos-razorpay-documentation-sjai branch 4 times, most recently from 5224e05 to 03cfdd9 Compare April 17, 2024 13:16
@sjai-odoo sjai-odoo marked this pull request as ready for review April 18, 2024 10:22
@C3POdoo C3POdoo requested a review from a team April 18, 2024 10:34
@qle-odoo
Copy link
Contributor

target 17.0 ?

@sjai-odoo
Copy link
Contributor Author

Hello @qle-odoo ,

Up to version 17.1, we had the option to directly select the terminal. However, starting from version 17.2 and subsequent versions, we now offer the choice between integrating with a terminal or using a QR code.

v-17.1
image

v-17.2 and subsequent versions
image

Therefore, we have included both options in the documentation to ensure that customers are not misled.
So we have created this in master after discussion with @mog-odoo

What's your call ?

Thank you.

@sjai-odoo sjai-odoo changed the base branch from master to 17.0 April 24, 2024 08:49
@sjai-odoo sjai-odoo force-pushed the master-pos-razorpay-documentation-sjai branch from 03cfdd9 to 95702a8 Compare April 24, 2024 09:18
@sjai-odoo
Copy link
Contributor Author

@qle-odoo
Branch retargeted to 17.0
Can you please check again
Thank you

@qle-odoo
Copy link
Contributor

@LoredanaLrpz can you review this please

@LoredanaLrpz LoredanaLrpz force-pushed the master-pos-razorpay-documentation-sjai branch from 95702a8 to 0269579 Compare July 22, 2024 07:49
Copy link
Contributor

@LoredanaLrpz LoredanaLrpz left a comment

Choose a reason for hiding this comment

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

I've made the changes and approved, but it still has to be changed in saas-17.2 --> you do not need to activate the razorpay module anymore; we'll have to replace line 34 with Activate Razorpay in the PoS application settings (with ref).

Also, when creating a PM, we have the Integration field now, (saas-17.2+)

@LoredanaLrpz LoredanaLrpz requested review from a team and removed request for a team July 22, 2024 08:13
@LoredanaLrpz LoredanaLrpz force-pushed the master-pos-razorpay-documentation-sjai branch 2 times, most recently from e9fb583 to 7772bd0 Compare July 22, 2024 08:28
@LoredanaLrpz LoredanaLrpz requested a review from a team July 22, 2024 08:29
Copy link
Contributor

@auva-odoo auva-odoo left a comment

Choose a reason for hiding this comment

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

Hi @sjai-odoo and @LoredanaLrpz thank you for your work! I left a few (minor) comments 🙂

Added the documentation needed to use razorpay payment method in Point Of Sale.

task- 3762059
@LoredanaLrpz LoredanaLrpz force-pushed the master-pos-razorpay-documentation-sjai branch from 7772bd0 to e6be388 Compare September 6, 2024 11:55
@LoredanaLrpz
Copy link
Contributor

@robodoo r+

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants