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#38 fix wording on event reg page #17695

Merged
merged 1 commit into from
Jul 14, 2020

Conversation

MegaphoneJon
Copy link
Contributor

https://lab.civicrm.org/dev/event/-/issues/38

Overview

The on-page instructions state "If you are registering additional people, you will be able to enter their registration information after you complete this page and click Continue", the only button available says 'Review you registration'.

https://civicrm.stackexchange.com/questions/35964/misleading-instructions-in-civievent-registration-template-for-multiple-particip

Before

Text says "Continue"

After

Text says "Review your registration".

Technical Details

lol

@civibot
Copy link

civibot bot commented Jun 25, 2020

(Standard links)

@yashodha
Copy link
Contributor

test this please

@seamuslee001
Copy link
Contributor

Jenkins re test this please

1 similar comment
@seamuslee001
Copy link
Contributor

Jenkins re test this please

@eileenmcnaughton
Copy link
Contributor

test this please

@eileenmcnaughton
Copy link
Contributor

@Stoob @agh1 if I can get a couple of thumbs up on this I will merge

Note no tested needed for trivial wording change

@Stoob
Copy link
Contributor

Stoob commented Jul 14, 2020

hurray for consistent wording

@eileenmcnaughton
Copy link
Contributor

test this please

@eileenmcnaughton
Copy link
Contributor

I got a couple of thumbs up from people with a good eye on usability & messaging - so MOP

Thanks @Stoob @kcristiano

@eileenmcnaughton
Copy link
Contributor

unrelated fail

@eileenmcnaughton eileenmcnaughton merged commit 868c96c into civicrm:master Jul 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants