-
-
Notifications
You must be signed in to change notification settings - Fork 812
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 Settings: rename labels for public listing, calendar links #31141
base: master
Are you sure you want to change the base?
Conversation
🤖 Thank you for contributing to CiviCRM! ❤️ We will need to test and review this PR. 👷 Introduction for new contributors...
Quick links for reviewers...
|
I also think the order is still wonky. Should be: You could even argue that where this is on the page is wonky. Really, Active and Public options should be at the top of the page. But that gets deeper. |
9e764db
to
ee42684
Compare
Good catch, thanks @guyiac ! Here is an updated version: |
jenkins, test this please |
TBH I stumble over the wording
wondering if something like below is more in keeping with the language style
|
ee42684
to
c705751
Compare
Definitely a typo, thanks @petednz ! How's this? On another note, I would have liked to remove the |
@bgm Thanks for working on this. I think that's much better. My suggestion to improve it further is to shorten the text and make them all statements rather than questions so they are consistent.
|
Or you could even separate the last three as follows:
Include with event information:
|
Overview
Renames some settings on Manage Event.
This are perhaps obvious to many experienced CiviCRM users, but I find that they systematically cause discomfort to admins during training.
Before
After