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

fix(links): Insert /f/<fileId> format for links to files #5412

Merged
merged 2 commits into from
Feb 27, 2024
Merged

Commits on Feb 26, 2024

  1. fix(links): Insert /f/<fileId> format for links to files

    Get rid of the old `/path/?fileId=<id>` format. It never really worked
    anyway because the paths were relative to the users' home directory.
    
    Instead, insert links to files/folders as full URLs with the
    `/f/<fileId>` format. This makes link previews work both in Text and
    Collectives.
    
    Rewrite links in the old format to the new format. But don't touch
    absolute link (without origin) to the collectives app to not break
    collectives-specific link handling to other pages.
    
    Signed-off-by: Jonas <[email protected]>
    mejo- committed Feb 26, 2024
    Configuration menu
    Copy the full SHA
    33dc25a View commit details
    Browse the repository at this point in the history
  2. fix(links): Always pass full URLs to NcReferenceList

    See discussions here for further information:
    * nextcloud-libraries/nextcloud-vue#5290
    * nextcloud-libraries/nextcloud-vue#5272
    
    Signed-off-by: Jonas <[email protected]>
    mejo- committed Feb 26, 2024
    Configuration menu
    Copy the full SHA
    ddb163b View commit details
    Browse the repository at this point in the history