Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] payment: process payments older than 10 minutes #29317

Closed
wants to merge 1 commit into from

Conversation

tbe-odoo
Copy link
Contributor

@tbe-odoo tbe-odoo commented Dec 6, 2018

  • The payment transaction processing cron retrieves all the transactions
    that need to be processed.

    In some cases, it can causes issues when trying to process payments
    that are being process by the payment polling page after returning
    from the payment acquirer.

    To avoid this concurrency issues, the cron will only process payments
    that have been validated for at least 10 minutes.

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

- The payment transaction processing cron retrieves all the transactions
  that need to be processed.

  In some cases, it can causes issues when trying to process payments
  that are being process by the payment polling page after returning
  from the payment acquirer.

  To avoid this concurrency issues, the cron will only process payments
  that have been validated for at least 10 minutes.
@KangOl
Copy link
Contributor

KangOl commented Dec 6, 2018

@robodoo r+

@robodoo robodoo added the r+ 👌 label Dec 6, 2018
@C3POdoo C3POdoo added the RD research & development, internal work label Dec 6, 2018
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Dec 6, 2018
robodoo pushed a commit that referenced this pull request Dec 6, 2018
- The payment transaction processing cron retrieves all the transactions
  that need to be processed.

  In some cases, it can causes issues when trying to process payments
  that are being process by the payment polling page after returning
  from the payment acquirer.

  To avoid this concurrency issues, the cron will only process payments
  that have been validated for at least 10 minutes.

closes #29317
@robodoo
Copy link
Contributor

robodoo commented Dec 6, 2018

Merged, thanks!

@robodoo robodoo closed this Dec 6, 2018
@fw-bot fw-bot deleted the 12.0-process-old-tx-fix-tbe branch October 20, 2019 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants