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

Status of test contribution is not fetched on ThankYou page. #13724

Merged
merged 2 commits into from
Mar 1, 2019

Conversation

agilewarealok
Copy link
Contributor

Overview : Steps to reproduce the bug

  1. Create a Contribution Page (Without confirm page) with Payment processor which captures the Payment instantly.
  2. Open the contribution page Test-Drive URL without logging into the CiviCRM
  3. Fill in the Billing and amount details
  4. Click on Submit
  5. Payment processor captures the payment instantly and user redirected on ThankYou Page.
  6. Confirmation message on ThankYou page is
    Your contribution has been submitted to XYZ

Even though the payment is already captured the confirmation message is misleading.

Before

Status of Test contribution is not fetched resulting in misleading confirmation message.

After

Correct status of test contribution is fetched so confirmation message is appropriate.

Comments

Agileware Ref: CIVIEWAY-42

@civibot
Copy link

civibot bot commented Feb 28, 2019

(Standard links)

@civibot civibot bot added the master label Feb 28, 2019
@agilewarealok
Copy link
Contributor Author

@eileenmcnaughton
Can you please review this ? PR we were discussing on MM.

@eileenmcnaughton
Copy link
Contributor

As discussed test doesn't exactly catch the bug but it does improve test cover & pave the way for next time. Bug fix = safe & sensible

@eileenmcnaughton eileenmcnaughton merged commit e4467bd into civicrm:master Mar 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants