[issue_tracker] Add ExternalIssueID field with permission control (#9795)#10254
[issue_tracker] Add ExternalIssueID field with permission control (#9795)#10254arnav-makkar wants to merge 1 commit intoaces:mainfrom
Conversation
|
@arnav-makkar Can you fix the tests on this PR? |
790a6b2 to
24698a5
Compare
|
@CamilleBeau Fixed the linter errors in |
|
Hi @CamilleBeau, could you please review this PR? |
| ALTER TABLE `issues` ADD COLUMN `externalIssueID` varchar(255) DEFAULT NULL AFTER `description`; | ||
|
|
||
| -- Add permission to view/edit external issue ID | ||
| INSERT INTO `permissions` (code, description, moduleID, categoryID) |
There was a problem hiding this comment.
The new permission should be given to admin
There was a problem hiding this comment.
Thank you for the feedback.
I've updated this SQL patch to grant the permission to the admin user now
|
@arnav-makkar could you fix the merge conflicts and then I will re-assign @CamilleBeau |
347fcf5 to
0ab37dd
Compare
|
Hi @skarya22, since this branch was a few months old I've rebased onto current main and resolved the merge conflicts. Admin (has the permission)Permission is granted on the admin account: The External Issue ID field renders on the Create New Issue form: Submitted an issue with a value in the field: On reopening, the value persists and the change is recorded in the comment history: Non-admin user (without
|






Brief summary of changes
This PR adds an External Issue ID field to the issue_tracker module, allowing administrators to link LORIS issues to external issue trackers (e.g., GitHub, Jira).
Key Features:
externalIssueIDfield (varchar 255) in theissuestableissue_tracker_external_idpermissionImplementation Details:
Database: Added column to
issuestable and updatedissues_historyenumBackend: Permission checks in GET (hide field) and POST (validate edits) endpoints
Frontend: Conditional rendering and editing based on user permission
Migration: SQL patch for existing databases (2026-01-04-Add_ExternalIssueID_to_issues.sql)
Have you updated related documentation?
Link(s) to related issue(s)