Skip to content

fix: HTR token seed migration#367

Merged
luislhl merged 2 commits intomasterfrom
fix/htr-token-seed-migration
Feb 24, 2026
Merged

fix: HTR token seed migration#367
luislhl merged 2 commits intomasterfrom
fix/htr-token-seed-migration

Conversation

@luislhl
Copy link
Collaborator

@luislhl luislhl commented Feb 11, 2026

Motivation

We were getting an error when running the seeds:

> yarn dlx sequelize-cli db:seed:all


Sequelize CLI [Node: 22.14.0, CLI: 6.6.5, ORM: 6.37.2]

Loaded configuration file "db/config.js".
Using environment "production".
== 20250416132150-add-htr-token: migrating =======

ERROR: Validation error

Acceptance Criteria

  • The seed migration that adds the HTR token should correctly set its version field.
  • The seed migration should run without errors on a clean DB.

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged
  • Make sure either the unit tests and/or the QA tests are capable of testing the new features
  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

@luislhl luislhl self-assigned this Feb 11, 2026
@luislhl luislhl moved this from Todo to In Progress (Done) in Hathor Network Feb 11, 2026
@luislhl luislhl moved this from In Progress (Done) to In Review (WIP) in Hathor Network Feb 12, 2026
@luislhl luislhl merged commit 2e942e0 into master Feb 24, 2026
1 check passed
@github-project-automation github-project-automation bot moved this from In Review (WIP) to Waiting to be deployed in Hathor Network Feb 24, 2026
@luislhl luislhl deleted the fix/htr-token-seed-migration branch February 24, 2026 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Waiting to be deployed

Development

Successfully merging this pull request may close these issues.

3 participants