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

Refresh Sessions or Login Functionality when expired sessions for receipt links & r #496

Closed
DevinWalker opened this issue Feb 26, 2016 · 6 comments
Assignees
Milestone

Comments

@DevinWalker
Copy link
Member

Scenario: A donor receives the confirmation email with the receipt link and clicks it after the expired time.

Problem: The donor sees the error message:

Sorry, it appears the viewing window for this donation receipt has expired or you do not have the permission to view this donation receipt.

This isn't very user friendly at all and is confusing and frustrating for donors. We need to give the donor the option to either log in or receive an updated link via the email used when they gave. Once they login, they should be properly redirected back to the receipt page.

@DevinWalker DevinWalker added this to the 1.3.7 milestone Feb 26, 2016
@DevinWalker DevinWalker self-assigned this Feb 26, 2016
@DevinWalker
Copy link
Member Author

Proposed fix:

Rather than You must be logged in to view this payment receipt. as seen here:

screen shot 2016-03-10 at 12 44 14 pm

  1. Add an additional option that states Don't have an account? Enter your email to be sent a new receipt will refresh the users session so when they click the receipt link it will actually work.

screen shot 2016-03-24 at 8 43 35 pm

1. Update the default "Donation Receipt" email to include more information than is included by default:

screen shot 2016-03-10 at 12 46 22 pm

Here's what it should look like:

screen shot 2016-03-10 at 12 51 14 pm

@DevinWalker
Copy link
Member Author

See: https://github.com/FacetWP/edd-no-logins for item 1 on the task list

DevinWalker pushed a commit that referenced this issue Mar 25, 2016
…o refresh access #496

Added options for the reCAPTCHA functionality
@DevinWalker DevinWalker modified the milestones: 1.3.7, 1.4 Mar 25, 2016
DevinWalker pushed a commit that referenced this issue Mar 25, 2016
Improved emails so they go through Give's email templates #496
DevinWalker pushed a commit that referenced this issue Mar 26, 2016
@DevinWalker
Copy link
Member Author

This feature is ready for testing.

How to tests:

  1. Enable the featured under Settings > Advanced
  2. Access the donation history page or receipt in a browser you are not logged in
  3. You should see the email form "Access Your Account" above.. enter your email
  4. You should receive an email with the access link, click it in the same browser you're not logged in
  5. You should see donation history and individual donation details

@DevinWalker
Copy link
Member Author

@mathetos see above

@mathetos
Copy link
Member

Just tested, one issue: when clicking on the donation history page where the form is, it immediately refreshes to a 404 and adds "undefined" into the URL. So far that is preventing me from testing further.

DEVIN EDIT - this was an invalid issue caused by theme's JS error

@DevinWalker
Copy link
Member Author

Another issue:

  • Links should always redirect to the Donation History URL to ensure there's always a payment_key when viewing the specific donation's details

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

No branches or pull requests

2 participants