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

EVENT: Cosmetic issue: Credit Card info on event confirmation page #217

Closed
magnolia61 opened this issue Dec 22, 2014 · 11 comments
Closed

EVENT: Cosmetic issue: Credit Card info on event confirmation page #217

magnolia61 opened this issue Dec 22, 2014 · 11 comments
Labels
Milestone

Comments

@magnolia61
Copy link
Contributor

I know it is just a cosmetic issue, but on the event confirmation page a line with Credit Card information shows, even when no CC PP at all is used. This might confuse some participants. It's probably an easy fix.

sepadd_master_confirmpage_ccinfo

@bjendres
Copy link
Member

@magnolia61 see #208 (comment)

@bjendres bjendres added this to the 1.1 milestone Jan 9, 2015
@bjendres
Copy link
Member

bjendres commented Jan 9, 2015

@magnolia61 Could you reproduce this again with the current master? And could you also please rule out the possibility that is has nothing to do with the SEPA extension?

@magnolia61
Copy link
Contributor Author

I doublechecked qwith the current master on a completely different website and it only happens when sepa_dd is enabled. The credit card information (and empty billing info) also pops up in the confirmation email btw.

sepadd_creditcard_info_email

@magnolia61 magnolia61 changed the title Cosmetic issue: Credit Card info on event confirmation page Event: Cosmetic issue: Credit Card info on event confirmation page Jan 12, 2015
@magnolia61 magnolia61 changed the title Event: Cosmetic issue: Credit Card info on event confirmation page EVENT: Cosmetic issue: Credit Card info on event confirmation page Jan 12, 2015
@bjendres bjendres assigned ghost Jan 12, 2015
@bjendres
Copy link
Member

@nikobo Could you have a look at this?

@magnolia61
Copy link
Contributor Author

The credit card information (and empty billing info) is redundant and needs to be removed from the event registration pages. On the other hand there some vital info sepamandate info is missing from

  1. confirmation screen
  2. thank-you screen and
  3. email confirmation
    When registering through a 'regular' contribution form the users' Bank account information and the mandate summary is (I must say brilliantly) displayed on screen.

In all cases the mandate-summary is missing from the email confirmation). For Dutch (and I guess also German and some users from some other EU countries) this is especially important as the thankyou screen and confirmation email are in fact the legal mandate that has just been given.

So, as well as removing the redundant creditcard (and empty billing info), the missing mandate info is also welcome to make it's way towards the event registration pages and confirmation email :-)

sepadd_donation_confirmation-screen_crop
sepadd_donation_thankyou-screen_crop

@tttp
Copy link
Member

tttp commented Jan 13, 2015

Trying to remember how I avoided it in the first version of the PP for
sepa.

@systopia, it was a mode/flag to set somewhere, but that created other
problems elsewhere that I had to work around.

I'll ping you and we have a conf to check out how to solve.

On 13 January 2015 at 09:39, Richard van Oosterhout <
notifications@github.com> wrote:

The credit card information (and empty billing info) is redundant and
needs to be removed from the event registration pages. On the other hand
there some vital info sepamandate info is missing from

  1. confirmation screen
  2. thank-you screen and
  3. email confirmation
    When registering through a 'regular' contribution form the users' Bank
    account information and the mandate summary is (I must say brilliantly)
    displayed on screen.

In all cases the mandate-summary is missing from the email confirmation).
For Dutch (and I guess also German and some users from some other EU
countries) this is especially important as the thankyou screen and
confirmation email are in fact the legal mandate that has just been given.

So, as well as removing the redundant creditcard (and empty billing info),
the missing mandate info is also welcome to make it's way towards the event
registration pages and confirmation email :-)

[image: sepadd_donation_confirmation-screen_crop]
https://cloud.githubusercontent.com/assets/2195908/5717914/e50fd21e-9b07-11e4-852c-eff056244c49.png
[image: sepadd_donation_thankyou-screen_crop]
https://cloud.githubusercontent.com/assets/2195908/5717913/e4152152-9b07-11e4-9fe8-d2a6ff3ac4ca.png


Reply to this email directly or view it on GitHub
#217 (comment).

@magnolia61
Copy link
Contributor Author

Would you advise me to split this issue in two seperate ones?

  1. issue Use sdd_creditor instead of inline parameters #1 remove credit card info on screens and in confirmation email
  2. issue Check Jo's comment about memberships missing a contribution_recur #2 include mandate summary info block in confirmation email

@bjendres
Copy link
Member

@magnolia61 Making the payment processor work for event registration pages is work in progress - we're actually on this right now. I'll ask you to test this again soon.

@bjendres
Copy link
Member

@magnolia61: I have now modified/extended the payment processor to work with event registrations. Could you please test with the current master?

@bjendres bjendres unassigned ghost Jan 15, 2015
@magnolia61
Copy link
Contributor Author

Thanks Bjorn for making the event registrations work. Your 'hack' to replace the credit card info on the confirmation and thank you screen works nice and registrations pass throught correctly now!
The credit card and billing info still appears in the confirmation email though. Probably that has to do with civicrm not realy understanding the type of payment processor civisepa actually is.

Possibly these are two really separate issues, but both deal with the confirmation mail:

  1. the missing mandate summary in confirmation email
  2. the reduntant CC+billing block) in the confirmation email

Both the billing block in the email, as well as the creditcard information have the following conditionals:
{if $contributeMode eq 'direct' and !$isAmountzero and !$is_pay_later and !$isOnWaitlist and !$isRequireApproval}

Would you think it would be possible to get the mandate summary information in the confirmation email and remove the billing & creditcard info? (without changing core?)

@bjendres
Copy link
Member

@magnolia61 Since the original problem of this ticket seems resolved, I moved this last topic to #237. Also, please note that I have currently no funding for the whole event registration, so I cannot spend arbitrary amounts of hours on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants