Skip to content
This repository was archived by the owner on Sep 9, 2024. It is now read-only.

Conversation

@CommanderStorm
Copy link
Contributor

Added the ?lfs=true flag to GitLab file requests
This means, that if a file is stored in LFS, return the content See decaporg/decap-cms#3704 for the upstream issue.
Any non-LFS Repos are not affected by this change. Here are the GitLab Docs on this feature to partially proof this.

I know that this is a bugfix, which is not on your roadmap (nor does it have an issue), but I think this should be an uncontroversial bugfix of the original project 😉

Screenshots

  • Previous Behaviour (with LFS enabled):
    image
  • Current Behaviour (with LFS disabled):
    image
  • Current Behaviour (with LFS enabled):
    image

This means, that if a file is stored in LFS, return the content
See decaporg/decap-cms#3704 for the upstream issue
@netlify
Copy link

netlify bot commented Jan 13, 2023

Deploy Preview for demo-staticjscms ready!

Name Link
🔨 Latest commit 020aa6a
🔍 Latest deploy log https://app.netlify.com/sites/demo-staticjscms/deploys/63c1abb64115a20008666a79
😎 Deploy Preview https://deploy-preview-346--demo-staticjscms.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Jan 13, 2023

Deploy Preview for staticjscms ready!

Name Link
🔨 Latest commit 020aa6a
🔍 Latest deploy log https://app.netlify.com/sites/staticjscms/deploys/63c1abb6d777d40008c0f144
😎 Deploy Preview https://deploy-preview-346--staticjscms.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@KaneFreeman KaneFreeman changed the title GitLab LFS Support feat: GitLab LFS Support Jan 14, 2023
@KaneFreeman KaneFreeman merged commit 41fd5fd into StaticJsCMS:main Jan 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants