nextcloud / calendar

📆 Calendar app for Nextcloud
https://apps.nextcloud.com/apps/calendar
GNU Affero General Public License v3.0
965 stars 237 forks source link

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

Open jancborchardt opened 6 years ago

jancborchardt commented 6 years ago

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!](https://www.bountysource.com/issues/49598425-show-proper-metadata-for-when-you-post-the-link-in-social-networks-or-chats?utm_campaign=plugin&utm_content=tracker%2F45525646&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F45525646&utm_medium=issues&utm_source=github).
tcitworld commented 6 years ago

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 commented 6 years ago

@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 commented 6 years ago

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 commented 6 years ago

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

georgehrke commented 4 years ago

Requires https://github.com/nextcloud/server/issues/17129