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

Sign-up link doesn't respect override from Flarum SSO plugin #25

Open
sptcguy opened this issue Apr 7, 2024 · 2 comments
Open

Sign-up link doesn't respect override from Flarum SSO plugin #25

sptcguy opened this issue Apr 7, 2024 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@sptcguy
Copy link

sptcguy commented Apr 7, 2024

In the Flarum SSO plugin, you can specify an override URL for the sign-up link/button to redirect users to a different site to sign up for an account.

The URL in the header changes successfully, however, this plugin does not respect the override. So the sign-up link presented by to the user in order to view a post triggers the default Flarum sign-up instead of redirecting them to our preferred location.

To reproduce:

  1. Install the Flarum SSO plugin.
  2. In the plugin settings, specify a new url for the sign-up link/button.
  3. Save
  4. Logout and navigate to a post.
  5. Click the sign-up button in the post area of the page.
@jslirola
Copy link
Owner

Hello, thanks for all the details to reproduce it.

I haven't had time to test it yet, but it makes sense that it doesn't work since the plugin requires some changes to make it compatible with the Flarum SSO plugin.

My availability is very limited, I'll let you know when I can take a look. In any case, collaborations are accepted and anyone can create a Pull Request to solve any issue.

@jslirola jslirola added enhancement New feature or request good first issue Good for newcomers labels Apr 12, 2024
@sptcguy
Copy link
Author

sptcguy commented Jun 14, 2024

A quick work-around I employed to hide the links is as follows:

  1. In admin area, go to "Appearance"
  2. Click "Edit custom CSS"
  3. And then put the following CSS override in:

.jslirolaLogin2seeplusRegister{ display: none;}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants