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

add link to element #356

Merged
merged 1 commit into from
Dec 4, 2024
Merged

add link to element #356

merged 1 commit into from
Dec 4, 2024

Conversation

merowin
Copy link
Collaborator

@merowin merowin commented Dec 3, 2024

I added the link to Element (for inviting people) - though I'm not sure if this is the best URL for this purpose.

@benjaminpjones
Copy link
Collaborator

though I'm not sure if this is the best URL for this purpose.

What does this mean? Is there another link we might consider? The invite link looks okay to me fwiw

@merowin
Copy link
Collaborator Author

merowin commented Dec 3, 2024

though I'm not sure if this is the best URL for this purpose.

What does this mean? Is there another link we might consider? The invite link looks okay to me fwiw

I don't know if for example this would be better or not: https://app.element.io/#/room/#govariants:matrix.org
I don't know the nuanced differences.

Copy link
Collaborator

@benjaminpjones benjaminpjones left a comment

Choose a reason for hiding this comment

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

Glad to have it!

@benjaminpjones
Copy link
Collaborator

if for example this would be better or not: https://app.element.io/#/room/#govariants:matrix.org

Oh gotcha, I have no idea. As long as it directs to the right place I guess either one is fine.

Copy link
Collaborator

@JonKo314 JonKo314 left a comment

Choose a reason for hiding this comment

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

I don't really know, but I think matrix.to is there to have nicer links, so this should be fine.

@merowin merowin merged commit 7b0e440 into main Dec 4, 2024
3 checks passed
@merowin merowin deleted the link_to_element branch December 4, 2024 16:23
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.

3 participants