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.

Sign up for GitHub

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

Jump to bottom

Incorrect donation notification - or possible email settings issue #38

Closed
rosswintle opened this issue Mar 16, 2015 · 2 comments
Closed

Incorrect donation notification - or possible email settings issue #38

rosswintle opened this issue Mar 16, 2015 · 2 comments
Assignees
@DevinWalker
Milestone
0.8 Beta

Comments

@rosswintle
Copy link

I seem to get the wrong email back to admin when I make a donation.

Donation notification settings:

screen shot 2015-03-16 at 21 39 31

Donation email:

screen shot 2015-03-16 at 21 40 54

Is this because I'm using "Default template" rather than "No template, plain text only"? And if so, should the fields for entering content be disabled if a template is selected rather than "No template, plain text only"?
@mathetos mathetos added the bug label Mar 16, 2015
@mathetos mathetos added this to the 0.8 Beta milestone Mar 16, 2015
@DevinWalker DevinWalker self-assigned this Mar 18, 2015
@DevinWalker
Copy link
Member

Related to #40 - once the default settings are being appropriately save on install this shouldn't be an issue. I've also updated the text within the default email so it's more aligned with donations.

DevinWalker pushed a commit that referenced this issue Mar 20, 2015
Default settings now save properly:
9c27c72 
Default values now populated for NEW installs only:

Success Page (page also created if value not set)
Failure Page
History Page

Base Country = US
Test Mode = Active
Gateways = Test Gateway

Admin Donation Email Notification = Default Set

Closes #40 and #38
@DevinWalker
Copy link
Member

Fixed with latest commit from 9c27c72

@mathetos mathetos mentioned this issue Sep 23, 2016
Closed
DevinWalker pushed a commit that referenced this issue Oct 7, 2016
@ramiy
Merge pull request #38 from WordImpress/release/1.7
e5055ae 
Merge updates
DevinWalker pushed a commit that referenced this issue Jun 30, 2017
@mehul0810
Merge pull request #38 from WordImpress/release/1.8.9
b75aca8 
Release/1.8.9
Genevieve-K added a commit to Genevieve-K/Give that referenced this issue Jan 19, 2023
@Genevieve-K
Fix Modal is not open after an error
de826ed 
With esc_url() escaping the redirect URL, special characters like &impress-org#38; as preserved, and the modal window is not open. As for preview messages, I suggest using esc_url_raw().
JoshuaHungDinh pushed a commit that referenced this issue Feb 6, 2023
@Genevieve-K
Fix Modal is not open after an error (#6677)
d5fbc8d 
With esc_url() escaping the redirect URL, special characters like & as preserved, and the modal window is not open. As for preview messages, I suggest using esc_url_raw().
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DevinWalker DevinWalker

Labels
None yet
Projects
None yet
Milestone
0.8 Beta
Development

No branches or pull requests
3 participants
@mathetos @rosswintle @DevinWalker