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

Community: Add Share drop down in community view. #1255

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

lubosz
Copy link
Contributor

@lubosz lubosz commented Oct 7, 2023

I found that there is no button to share a community url. This patch fixes that.
Not sure if this is the best way to create a community URL, let me know if I am missing something.
Also not sure if https can be expected from all instances. Didn't find any places were instance / community URLs are already created.

Copy link
Collaborator

@MV-GH MV-GH left a comment

Choose a reason for hiding this comment

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

There is a example in ParseUrl. It does look like we assume https from all instances. I don't really see away to check it anyway. And Android forbids most http traffic anyway due to ClearText.

Copy link
Member

@dessalines dessalines left a comment

Choose a reason for hiding this comment

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

Thx!

@dessalines dessalines merged commit e46abcd into LemmyNet:main Oct 17, 2023
1 check 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.

3 participants