server icon indicating copy to clipboard operation
server copied to clipboard

Backend for reference metadata fetching

Open juliusknorr opened this issue 3 years ago • 0 comments

  • [ ] Frontend integration https://github.com/juliushaertl/vue-richtext/pull/743
  • [ ] Requires https://github.com/nextcloud/3rdparty/pull/1131

ToDo

Backend

  • [ ] how to handle images (proxy/caching)
  • [ ] caching per user or reference
  • [ ] cache ttl per reference
  • [ ] decide if storing references in the db instead
  • [ ] rate limit for uncached references?
  • [ ] config option to disable opengraph

Frontend

  • [ ] check on image sizing for open graph
  • [ ] fallback on small images
  • [ ] Expose widget API
  • [ ] Design polishing
  • [ ] Files widget

juliusknorr avatar Aug 10 '22 08:08 juliusknorr

It took me so long to understand what this is about, so, for other unaware reviewers: This is about showing a preview of links posted in Talk, like what is done when posting links on other chat platforms.

And this is the backend part, is not specific to Talk, and may not be specific to links in the future, which is what makes the original description convoluted.

come-nc avatar Aug 29 '22 09:08 come-nc

Sorry, should have clarified that a bit more, but I've taken your description for the first post in addition

juliusknorr avatar Aug 29 '22 09:08 juliusknorr

HEAD detached at pull/33494/merge Changes not staged for commit: (use "git add ..." to update what will be committed) (use "git restore ..." to discard changes in working directory) modified: dist/core-main.js.map

Recompile?

nickvergessen avatar Aug 31 '22 12:08 nickvergessen

Failures unrelated

juliusknorr avatar Aug 31 '22 18:08 juliusknorr