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: include receipt_link_url in plain text emails #4284

Closed
1 of 3 tasks
mehul0810 opened this issue Oct 23, 2019 · 1 comment · Fixed by #4285
Closed
1 of 3 tasks

fix: include receipt_link_url in plain text emails #4284

mehul0810 opened this issue Oct 23, 2019 · 1 comment · Fixed by #4285
Assignees

Comments

@mehul0810
Copy link
Contributor

@mikejhale commented on Wed Oct 23 2019

Bug Report

User Story

As a donor, I want have a link to my PDF receipt included in my donation email so that I have a record of my donation.

They are using text emails instead of HTML. The {receipt_link_url} isn't working. Clicking on it turns absolutely no result for me. For them, they get the error message: "The specified receipt ID appears to be invalid."

They excluded rules in caching plugin, disabled all plugins, installed Native PHP Session, added "define( 'GIVE_USE_PHP_SESSIONS' , false);" to wp-config.php, removed Donation Confirmation and History pages from caching, disabled Lightspeed Caching plugin, excluded the form from caching and still see the error.

Current Behavior

receipt_link_url is blank in plain text emails

Expected Behavior

receipt_link_url is in plain text emails is a valid link to the PDF

Bug Type

  • I am not sure whether this functionality ever worked as expected.

Steps to Reproduce

  1. Turn emails to Plain intead of HTML
  2. Add pdf_receipt email tag to the email
  3. Enable PDF receipts addon
  4. Submit a donation
  5. Click the PDF receipts link in the email

Related

https://give.productboard.com/feature-board/879919-givewp-feature-hierarchy/features/2927326/detail

Acceptance Criteria

  • receipt_link_url is a valid link in plain text emails
  • receipt_link_url is a valid link in HTML emails

@mehul0810 commented on Wed Oct 23 2019

@mikejhale @DevinWalker On doing research on this issue, I noticed that this issue is related to Give core. So, moving this issue to Give core and creating PR for core.

@mikejhale
Copy link
Contributor

tested:
-[x] receipt_link_url is a valid link in plain text emails

mikejhale added a commit that referenced this issue Oct 23, 2019
fix: include receipt_link_url in plain text emails #4284
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

Successfully merging a pull request may close this issue.

2 participants