Skip to content

Migrate to Pydantic 2 for LNbits 1.7.0 - #20

Draft
motorina0 wants to merge 1 commit into
mainfrom
pydantic2
Draft

motorina0 wants to merge 1 commit into
mainfrom
pydantic2

Conversation

@motorina0

Copy link
Copy Markdown
Collaborator

Makes Build your own! compatible with the Pydantic 2 models in LNbits 1.7.0 and sets min_lnbits_version to 1.7.0. Updates validation, serialization, and current LNbits API usage where needed.

Adds the missing config.json version (1.0.6) from the latest entry in the extension registry.

Depends on the core Pydantic 2 migration: lnbits/lnbits#4253.

Validation used the local LNbits Pydantic 2 checkout and FakeWallet:

  • make all passed.
  • make test: no tests collected (pytest exit 5).
  • Authenticated home page returned HTTP 200 and rendered in the browser.
  • Config JSON validation and git diff --check passed after the metadata updates.

The published LNbits package was not used for these checks; this PR targets the upcoming 1.7.0 core.

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.

1 participant