Skip to content

feat: add configurable issue url#7734

Open
sempostma wants to merge 3 commits intodecaporg:mainfrom
laikacms:feat/custom-issue-url
Open

feat: add configurable issue url#7734
sempostma wants to merge 3 commits intodecaporg:mainfrom
laikacms:feat/custom-issue-url

Conversation

@sempostma
Copy link

Summary

A small change that makes the issue url configurable.

In my case, I need this since my client is non-technical. Redirecting them to a Github issue page will just confuse them. A custom url will allow for creating a custom endpoint for automatic reporting of issues.

@martinjagodic
Copy link
Member

Great idea @sempostma.

Please also add this to the schemas, as it is a new config field. These files all generate a schema (don't ask me why there's 3):

  • packages/decap-cms-core/index.d.ts
  • packages/decap-cms-core/src/constants/configSchema.js
  • packages/decap-cms-core/src/types/redux.ts

We will also need a PR on the docs site that documents the new option. I think we should put it above Collections on https://decapcms.org/docs/configuration-options/

@sempostma sempostma requested a review from a team as a code owner February 26, 2026 11:11
@sempostma
Copy link
Author

I've updated the schema and added a PR for the docs: decaporg/decap-website#152

martinjagodic
martinjagodic previously approved these changes Feb 26, 2026
@martinjagodic martinjagodic enabled auto-merge (squash) February 26, 2026 12:02
auto-merge was automatically disabled March 5, 2026 08:05

Head branch was pushed to by a user without write access

@sempostma sempostma force-pushed the feat/custom-issue-url branch from 213a913 to e497548 Compare March 5, 2026 08:05
@sempostma
Copy link
Author

@martinjagodic Rebased on the main branch to fix the minimatch error.

yanthomasdev
yanthomasdev previously approved these changes Mar 5, 2026
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.

Code LGTM, I'll review the docs next, appreciate the contribution 🙌

@martinjagodic
Copy link
Member

@sempostma, please run npm run format so tests will pass. thanks!

@sempostma sempostma dismissed stale reviews from yanthomasdev and martinjagodic via 54cba3b March 6, 2026 08:51
Copy link
Member

@martinjagodic martinjagodic left a comment

Choose a reason for hiding this comment

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

Thanks. I am fixing tests in another PR. When it's merged, this one follows soon after

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