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

Update the matrix link to join the space #145

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

altf4arnold
Copy link

Clicking on the link in the current state will ask to log in with a username hosted on chat.gnuradio.org. Additional manipulation is required to add it to a matrix client (when a person already has one set).

The new version of the link will propose to either create an account/install an app from scratch or will just add itself to the already existant matrix client of the user

content/contact/_index.md Outdated Show resolved Hide resolved
@marcusmueller
Copy link
Member

Note that I think it's the right approach to drop people directly to the space, please make that change, just the detour through matrix.to is negative :)

ERROR deprecated: .Site.DisqusShortname was deprecated in Hugo v0.120.0 and will be removed in Hugo 0.137.0. Use .Site.Config.Services.Disqus.Shortname instead.
@altf4arnold
Copy link
Author

Ok. So apparently, the issue is a little bit deeper. Turns out that the link on the website was already good enough. It just wasn't visible (unless you know it is there).

So instead of changing the links, I changed the CSS to make it stand out more.

I also got a deprecation error at build time. So I also changed that

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.

2 participants