jason/breedr
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Compare commits

base: jason:9de792aa02c03613e3a32b880107a7de1926597c
Branches Tags
jason:main jason:main2
..
compare: jason:4e5b695c2258cfcf195b2b823bdc551caba5da34
Branches Tags
jason:main jason:main2

2 Commits
9de792aa02 ... 4e5b695c22

Author SHA1 Message Date
jason
4e5b695c22 Merge pull request 'fix(db): add vet_name, result, next_due to healthMigrations ALTER TABLE guards' (#38) from fix/health-records-missing-columns into master 
Reviewed-on: #38
 2026-03-10 12:52:40 -05:00
jason
9b43bdab99 fix(db): add vet_name, result, next_due to healthMigrations ALTER TABLE guards 
Existing databases were missing these 3 columns because they were defined
in the CREATE TABLE but not in the healthMigrations array used for
ALTER TABLE on pre-existing DBs. This caused the 'table health_records
has no column named vet_name' error in the Add Health Record modal.
 2026-03-10 12:52:04 -05:00
1 changed files with 4 additions and 1 deletions
Expand all files Collapse all files

5
server/db/init.js
View File

@@ -128,7 +128,7 @@ function initDatabase() {
) )
`); `);
// migrate: add OFA-specific columns if missing // migrate: add OFA-specific columns if missing (covers existing DBs)
const healthMigrations = [ const healthMigrations = [
['test_type', 'TEXT'], ['test_type', 'TEXT'],
['ofa_result', 'TEXT'], ['ofa_result', 'TEXT'],
@@ -136,6 +136,9 @@ function initDatabase() {
['performed_by', 'TEXT'], ['performed_by', 'TEXT'],
['expires_at', 'TEXT'], ['expires_at', 'TEXT'],
['document_url', 'TEXT'], ['document_url', 'TEXT'],
['result', 'TEXT'],
['vet_name', 'TEXT'],
['next_due', 'TEXT'],
]; ];
for (const [col, def] of healthMigrations) { for (const [col, def] of healthMigrations) {
try { db.exec(`ALTER TABLE health_records ADD COLUMN ${col} ${def}`); } catch (_) { /* already exists */ } try { db.exec(`ALTER TABLE health_records ADD COLUMN ${col} ${def}`); } catch (_) { /* already exists */ }