Telegram (ask questions / claim the issue here first): https://t.me/+DOylgFv1jyJlNzM0
Why this matters
migrations/1788000000018_add-loan-events-missing-indexes.js:43 defines down = () => {} (no-op), so rolling this migration back leaves idx_loan_events_borrower, idx_loan_events_event_type, idx_loan_events_loan_id_event_type and idx_loan_events_created_at behind. A re-apply or a later differently-defined index of the same name then collides, and the migration is not truly reversible.
Acceptance criteria
Files to touch
backend/migrations/1788000000018_add-loan-events-missing-indexes.js
Out of scope
- Colliding migration timestamp prefixes (already filed)
Why this matters
migrations/1788000000018_add-loan-events-missing-indexes.js:43 defines down = () => {} (no-op), so rolling this migration back leaves idx_loan_events_borrower, idx_loan_events_event_type, idx_loan_events_loan_id_event_type and idx_loan_events_created_at behind. A re-apply or a later differently-defined index of the same name then collides, and the migration is not truly reversible.
Acceptance criteria
Files to touch
backend/migrations/1788000000018_add-loan-events-missing-indexes.jsOut of scope