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

Member-only donation forms don't display member-only validation responses properly #551

Closed
DevinWalker opened this issue Mar 30, 2016 · 5 comments
Assignees
Milestone

Comments

@DevinWalker
Copy link
Member

Scenario: When "Disable Guest Donations" is enabled and "Register / Login Form" set to None for a particular donation form and a guest attempts to make a donation - this is called a "Member-only" donation form.

screen shot 2016-03-30 at 10 47 43 am

Problem: The form just reloads without any indication that it's only a members donation form

How to fix? Options:

  1. Close the form entirely and display a message above that the form is for member donations only
  2. Don't close the form at all, when a guest tries to donate display an error message that indicates it's only for members

@mathetos thoughts?

@DevinWalker DevinWalker added this to the 1.4.1 milestone Apr 4, 2016
@DevinWalker DevinWalker self-assigned this Apr 7, 2016
@DevinWalker DevinWalker added the bug label Apr 7, 2016
DevinWalker pushed a commit that referenced this issue Apr 8, 2016
DevinWalker pushed a commit that referenced this issue Apr 8, 2016
…orm meta option rather #551

New param $form_id in give_logged_in_only()
@DevinWalker DevinWalker changed the title Member-only Donation Forms Reloading Improperly When Guest Donates Member-only donation forms don't display member-only validation responses properly Apr 8, 2016
DevinWalker pushed a commit that referenced this issue Apr 8, 2016
@DevinWalker DevinWalker mentioned this issue Apr 8, 2016
@mathetos
Copy link
Member

Hmmm... I really don't like putting the burden of discovering a setting problem on the DONOR instead of on the Admin. The error should appear in the Form Edit screen so that the Admin can fix it before it's live for donors.

@mathetos mathetos reopened this Apr 11, 2016
@DevinWalker
Copy link
Member Author

What do you mean setting problem? It's not a problem. It's creating a member-only donation form that can be used for various reasons. If the admin places it publicly and non-logged in users can access it then this is where error appears

@mathetos
Copy link
Member

Yes, which means that a non-logged in donor attempts to donate and only after filling in all the information gets a message that tells them they can't donate. Not a good donor-experience.

@DevinWalker
Copy link
Member Author

So... #1 Close the form entirely and display a message above that the form is for member donations only

@mathetos
Copy link
Member

Ya, I think that's far better for the donor overall. And I can see that if the Admin is using a membership plugin of some sort then the combo of "Disable Guest Donations" and "No login/registration" form would actually be beneficial for them.

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