-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some questions added
README.md
Outdated
1. The user confirms the payment in the first tab | ||
1. Paypal has the address of Berlin, but the shop will deliver the goods to Paris. | ||
|
||
**Possible solution:** the backend calls `patch/payments` endpoint every time user changes it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- backend or frontend calls patch?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be backend:
* The frontend - the browser part of the shop. This is what the user interacts with.
* The backend - the shop server.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, but how backend get known the a changes the address? Only front-end could know this and call the patch, isn't it?
README.md
Outdated
1. user has cart with item1=10€ | ||
1. user is redirected to Paypal payment page | ||
1. in a different tab, user changes his cart to e.g. item2=20€ | ||
1. user completes the payment in the first tab |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and redirects back to approval page
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I put the shop
instead of approval page
. 865ce6b
README.md
Outdated
1. user is redirected to Paypal payment page | ||
1. in a different tab, user changes his cart to e.g. item2=20€ | ||
1. user completes the payment in the first tab | ||
1. paypal approves the payment for 10€, but the real total amount has changed to 20€ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
real total amount of sold (shipped) items
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
README.md
Outdated
1. user completes the payment in the first tab | ||
1. paypal approves the payment for 10€, but the real total amount has changed to 20€ | ||
|
||
**Possible solution:** the backend has to compare total amount of the payment and total amount of payment's cart before calling `patch/payments` endpoint. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
before calling execute
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixes #62