Skip to content

Comments

Update page Decap Proxy#140

Open
martinjagodic wants to merge 4 commits intomainfrom
proxy-update
Open

Update page Decap Proxy#140
martinjagodic wants to merge 4 commits intomainfrom
proxy-update

Conversation

@martinjagodic
Copy link
Member

Explain changes from decaporg/decap-cms#7579

@netlify
Copy link

netlify bot commented Oct 16, 2025

Deploy Preview for decap-www ready!

Name Link
🔨 Latest commit 4172dfd
🔍 Latest deploy log https://app.netlify.com/projects/decap-www/deploys/6985f122201228000881a81e
😎 Deploy Preview https://deploy-preview-140--decap-www.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 project configuration.

Copy link

@palant palant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good apart from the one minor issue.

There is a more fundamental issue here but it isn’t a new one: this piece of documentation assumes that the repository isn’t a local checkout that needs to be synced up with a remote server. Arguably, modifying the primary repository directly should be an uncommon setup.

It is possible to add a post-commit hook that will automatically push the changes committed by Decap Proxy, yet pulling prior to commit doesn’t seem to be doable with hooks. I had to solve this by modifying my version of Decap Proxy but I’m not sure what a good general solution would look like.

@martinjagodic
Copy link
Member Author

Thanks for your review.

Updating the local folder is the intended behavior. I also changed the wording because of this. Before, docs said local repository, now it's local folder.

However, we could have a flag that would update the remote repository.

Copy link
Contributor

@yanthomasdev yanthomasdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work here @martinjagodic, I have a few suggestions.

---
group: Accounts
weight: 70
title: "Decap Proxy: Work With a Local Repository"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a big title, in documentation subtitles shouldn't appear.

I would probably change it to "Local Backend" (since it is directly related to the setting), what do you think? Note that SEO and search will look into the contents where we specify we are talking about a local folder.

Suggested change
title: "Decap Proxy: Work With a Local Repository"
title: "Local Backend"

Alternatively, "Decap Proxy" is better

Suggested change
title: "Decap Proxy: Work With a Local Repository"
title: "Decap Proxy

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The feature is actually called "Decap Server", but if this is the title, it can be confusing, as it's only intended for local use. So we could have

  • Decap Server for Local Development
  • Decap Server (Local Backend)
  • Local Development with Decap Server
  • ...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's generally indicative that the feature name itself could use a rename! With that considered, I guess Local Development fits best.

martinjagodic and others added 2 commits February 6, 2026 14:46
Co-authored-by: Yan <61414485+yanthomasdev@users.noreply.github.com>
Co-authored-by: Yan <61414485+yanthomasdev@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants