feat: Add full customization for GitHub integration + improve task linking logic #636
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This adds a new environnement variable KANEO_DISABLE_BRANDING to disable Kaneo's branding in the github integration. In the future, this could also be used for the white labeling issue discussed in the discord #suggestion channel.|This PR adds full customization for the GitHub integration, allowing users to define the issue title, description, and the automatic comment posted when an issue is created in GitHub and imported into kaneo.
It also fixes a bug where switching between projects (one with an integration and one without) caused the form to retain values instead of resetting properly.
Additionally, the hard-coded “kaneo” tag has been removed, as it provided no real value; in the future we could introduce customizable tags if needed.
Finally, the logic for determining whether a task already exists when receiving data from the GitHub webhook has been improved by using a new linked_issue_id column rather than relying on title.
Related Issue(s)
Fixes #628
Type of Change
These new features are not a breaking change. Since we add Title Template and Description Template empty, those will use the same template that we currently use. For the Comment Template, I've created two migration. One that adds the column with the current message as a default, and another one that drops the default.
How Has This Been Tested?
Screenshots (if applicable)
Checklist
Additional Notes