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 chat information #2106

Merged
merged 1 commit into from
Nov 1, 2023
Merged

Update the chat information #2106

merged 1 commit into from
Nov 1, 2023

Conversation

ares
Copy link
Member

@ares ares commented Oct 18, 2023

No description provided.

support.md Outdated

In addition to the [general guidelines](/support.html#CommunityGuidelines) above, we also have a few IRC-specific guidelines:
In addition to the [general guidelines](/support.html#CommunityGuidelines) above, we also have a few Matrix-specific guidelines:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd s/Matrix/chat/ here, as the protocol does not influence the guidelines

support.md Outdated
A great place to get real-time help with The Foreman is our IRC channel:

* **#theforeman** on [libera.chat](https://libera.chat/)
## ![Matrix icon](static/images/matrix.png) [Matrix.org live chat](https://riot.im/app/#/room/#theforeman:matrix.org)
Copy link

@omenos omenos Oct 19, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be better to use the more generic URL for the matrix rooms?

https://matrix.to/#/#theforeman:matrix.org

Or updating to use the Element domain:

https://app.element.io/#/room/#theforeman:matrix.org

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These takes me through 2 more screen though, right? I like it mentions more matrix clients. Any other benefit? (I'm not opposed to it)

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The simple change would just be to use the new Element URL rather than the legacy Riot domain. The matrix.to share link lets the user access the room with the client of their choice, but it does involve a few extra clicks if they don't already have an app or web session ready to go.

Either way, a user can't preview the room/discussion until they have an account on some server. While linking directly to the room makes sense, using matrix.to makes sense from the Matrix community perspective of showcasing the decentralized nature of the platform. Whether or not that will just be more confusing for new non-Matrix users... I'll leave that to someone else to determine as I am a biased source.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

makes sense now, thanks, I've updated the domain but left the direct link to the room in element, as that's the fastest way to get involved, people can find better client or get an account on another server later.

support.md Outdated Show resolved Hide resolved
@ares
Copy link
Member Author

ares commented Oct 31, 2023

Updated, ready for re-review

Copy link

@omenos omenos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ekohl ekohl merged commit 40b5e97 into theforeman:gh-pages Nov 1, 2023
2 checks passed
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.

4 participants