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

Show proper metadata for when you post the link in social networks or chats #609

Open
jancborchardt opened this issue Sep 21, 2017 · 5 comments
Labels
1. to develop Accepted and waiting to be taken care of enhancement New feature request good first issue Small tasks with clear documentation about how and in which place you need to fix things in.

Comments

@jancborchardt
Copy link
Member

jancborchardt commented Sep 21, 2017

Currently you see stuff like this:
calendar link js
»This app requires Javascript…« instead of displaying the correct Calendar metadata, like the name and the person it was shared by, which should be in the website metadata (ogp or so).

cc @nextcloud/calendar


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@jancborchardt jancborchardt added 1. to develop Accepted and waiting to be taken care of bug good first issue Small tasks with clear documentation about how and in which place you need to fix things in. labels Sep 21, 2017
@tcitworld
Copy link
Member

It's strange that Discourse (?) doesn't uses the OGP tags we have. It works just fine in Facebook at least.
Maybe we could add Twitter tags too.

@jancborchardt
Copy link
Member Author

jancborchardt commented Sep 22, 2017

@tcitworld that’s actually Riot: https://riot.im – maybe sign up there and test it, could be related to issues with others? Maybe Discourse actually has similar problems. Cause for other sites it works without issue.

@tcitworld
Copy link
Member

It seems Matrix (which Riot is based upon) tries to get some kind of description since we don't provide one through the OCP tags : https://github.com/matrix-org/synapse/blob/master/synapse/rest/media/v1/preview_url_resource.py#L408
Maybe we could have a default one ? The files_sharing app uses %filename% is publicly shared, I guess we could add the same.

@georgehrke
Copy link
Member

@tcitworld Would you like to look into this issue? :)

@georgehrke
Copy link
Member

Requires nextcloud/server#17129

@ChristophWurst ChristophWurst modified the milestones: 2.2.0, 2.3.0 Mar 24, 2021
@ChristophWurst ChristophWurst modified the milestones: v2.3.0, v2.4.0 Jun 24, 2021
@ChristophWurst ChristophWurst modified the milestones: v2.4.0, v2.4.1 Nov 25, 2021
@tcitworld tcitworld removed this from the v2.4.1 milestone Dec 17, 2021
@miaulalala miaulalala added enhancement New feature request and removed bug labels Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of enhancement New feature request good first issue Small tasks with clear documentation about how and in which place you need to fix things in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants