Skip to content

Conversation

@mmsinclair
Copy link
Contributor

@mmsinclair mmsinclair commented Sep 2, 2025

This PR removes sqlite support, requiring pgsql to run the NS API.

It also fixes the following issues:

  • deserialisation of NodeDescription
  • defaults for WireguardDetails for deserialisation

This change is Reviewable

@mmsinclair mmsinclair requested a review from durch September 2, 2025 23:31
@vercel
Copy link

vercel bot commented Sep 2, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
nym-explorer-v2 Ready Ready Preview Comment Sep 18, 2025 6:38pm
nym-node-status Ready Ready Preview Comment Sep 18, 2025 6:38pm
1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
docs-nextra Ignored Ignored Preview Sep 18, 2025 6:38pm

Copy link
Contributor

@durch durch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only nit I have is the versioning, this chopping is a major semantic version.

ns-api: add env var to skip migrations for local dev

ns-api: tidy up imports

ns-api: fix deserialisation fo node descriptions

update dockerfile

update README

fix up README and example env

ns-api: bump major version to 4

ns-api: add more geoip data and new performance field in dvpn responses
@mmsinclair
Copy link
Contributor Author

The only nit I have is the versioning, this chopping is a major semantic version.

Bumped to v4.0.0

dynco-nym and others added 2 commits September 17, 2025 16:13
* Use explicit transaction for testrun status change

* Improve run scripts

* Skip locked rows
@mmsinclair mmsinclair merged commit fb0b55d into develop Sep 19, 2025
14 of 17 checks passed
@mmsinclair mmsinclair deleted the mark/ns-api-pg-only branch September 19, 2025 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants