wdevs/amcs
5
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 3 Wiki Activity

fix: correct chat_histories FK to reference projects(guid) #3

Merged
warkanum merged 2 commits from fix/chat-history-fk into main 2026-04-01 14:29:10 +00:00
Conversation 2 Commits 2 Files Changed 1
+1 -1
sam commented 2026-04-01 14:25:35 +00:00
Member
Copy Link
Copy Source

Fixes #2

Problem

chat_histories.project_id was referencing projects(id) which is a bigserial, causing an error when trying to insert with a UUID project reference.

Fix

  • migrations/019_fix_chat_histories_fk.sql — drops and recreates the FK constraint pointing to projects(guid) on existing deployments
  • migrations/018_chat_histories.sql — corrected for fresh installs
Fixes #2 ## Problem `chat_histories.project_id` was referencing `projects(id)` which is a `bigserial`, causing an error when trying to insert with a UUID project reference. ## Fix - `migrations/019_fix_chat_histories_fk.sql` — drops and recreates the FK constraint pointing to `projects(guid)` on existing deployments - `migrations/018_chat_histories.sql` — corrected for fresh installs
sam added 1 commit 2026-04-01 14:25:35 +00:00
fix: correct chat_histories FK to reference projects(guid) f163b9c370 
Fixes #2 — project_id on chat_histories was referencing projects(id)
(bigserial) instead of projects(guid) (uuid). Added migration 019 to
repair existing deployments and corrected 018 for fresh installs.
warkanum requested changes 2026-04-01 14:28:15 +00:00
warkanum left a comment
Owner
Copy Link
Copy Source

No need for the fix in migrations/019_fix_chat_histories_fk.sql . I dropped the bugged table manually.

No need for the fix in migrations/019_fix_chat_histories_fk.sql . I dropped the bugged table manually.
sam added 1 commit 2026-04-01 14:28:47 +00:00
fix: remove migration 019, table was dropped manually 9407c05535
sam commented 2026-04-01 14:28:51 +00:00
Author
Member
Copy Link
Copy Source

Removed migration 019 — since the table was dropped manually, 018_chat_histories.sql (now with the correct projects(guid) FK) will recreate it cleanly on the next migration run.

Removed migration 019 — since the table was dropped manually, `018_chat_histories.sql` (now with the correct `projects(guid)` FK) will recreate it cleanly on the next migration run.
warkanum merged commit 24532ef380 into main 2026-04-01 14:29:10 +00:00
warkanum deleted branch fix/chat-history-fk 2026-04-04 12:18:41 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
warkanum
Labels
Clear labels
architecture
debugging
integration
planned
ux
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
hein (Hein) root sam sgcommand (SG Command) warkanum (Warky)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: wdevs/amcs#3