Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

WP.paymentStatus getting updated before WPP.status display for bulk payments #47

Open
sandhiyakavi opened this issue Jun 23, 2021 · 3 comments
Labels
P3 wontfix This will not be worked on

Comments

@sandhiyakavi
Copy link
Collaborator

Description:

I am able to view in some cases WP.paymentStatus getting updated to 'completed' even before the WPP.status is changed to 'completed' (still in 'in-progress') status while performing bulk payments.

status_completed_before_inside_changed

@maxceem
Copy link
Contributor

maxceem commented Jun 23, 2021

@sandhiyakavi This is fixed now together with topcoder-platform/taas-apis#213.

It was issue in TaaS Api. It should be fixed for existent and for new payments.

@sandhiyakavi
Copy link
Collaborator Author

@maxceem I verified this issue. I am still able to view WPP.status getting updated before WP.paymentstatus and hence sometimes it's like it is providing us wrong status.
Note: If we refresh again it is also updated and showing correct status. The problem here is since we need to refresh each time to know the updated status and WPP.status sometimes updating before WP.paymentstatus

WP status changing before

Wpstatus_updating_faster

@maxceem
Copy link
Contributor

maxceem commented Jun 24, 2021

@sandhiyakavi unfortunatelly it's not that easy to fix this issue at the moment. There indeed could be deplay between statuses update and paymentStatus might get updated first before the status of corresponding payment.

The internal reason for this is caused by the fact that we send Kafka events to ES Processor, but order of which these events would be processed is not guaranteed for different topics.
So this is the same kind of issue, as if we process payment, reload page, but possible could still get old status instead of in-progress.

I guess the only solution is to merge ES Processor into TaaS API which is a big change. So I would say this that this could be listed as a know issue which we cannot fix for now. Though I don't think it's very critical, as most likely we don't have to open payments popup immediately and can track payment status using main status in the table.

cc @nkumar-topcoder

@sandhiyakavi sandhiyakavi changed the title Inconsistency between WP.paymentStatus and WPP.status display for bulk payments WP.paymentStatus getting updated before WPP.status display for bulk payments Jun 28, 2021
@maxceem maxceem added the wontfix This will not be worked on label Jun 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
P3 wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants