-
Notifications
You must be signed in to change notification settings - Fork 29
Fix roster role link in members, and remove role description top-level link in public. #1332
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
Fix roster role link in members, and remove role description top-level link in public. #1332
Conversation
|
I agree that role descriptions featuring in the main nav is more prominent than they need to be. |
4ae441f to
f18b6c6
Compare
Yes - the use case where the members area is disabled, or people aren't in the habit of using it. When people hit a login page they're not familiar with, x% of them give up. So my church uses public roster URLs, with the "public roster secret" setting to make it less world-findable.
Yes, that last example is what I had in mind (with subtitle "roster role descriptions"). But the single link would also be OK. |
|
I chuckled when I just came across this: #1217 |
…ay Roster' menu to 'Rosters'. Fixes #1331
Co-authored-by: Andrew Murray <[email protected]>
f18b6c6 to
ff1379a
Compare
|
Replaced by PR #1346 |



Fixes #1330 and #1331 (changes are intedependent).
#1330 is a simple fix:
BASE_URL.'/public/?view=display_role_description...to?view=_roster_role_descriptioninroster_view.class.public/equivalent, for the now-relative hyperlink to point to.#1331 is also a simple but messy business of renaming files and links.
The result: