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

Correct text for membersonly site and disabled login #5333

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Jimmi08
Copy link
Contributor

@Jimmi08 Jimmi08 commented Sep 2, 2024

Motivation and Context

#4349

Description

This change respects selected user registration system and correct message for membersonly site.

  • if login is disabled (0), there is no login link
    This is a restricted area.
  • if only login is allowed, there is login link but with dot (finished sentence) -
    This is a restricted area. For access please [log in].
  • if login and signup allowed, there is original solution
    This is a restricted area. For access please [log in] or [register] as a member.

How Has This Been Tested?

Types of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation (a change to man pages or other documentation)

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

Successfully merging this pull request may close these issues.

1 participant