Skip to content

Feat: add fiat denominated withdraw links #71

Merged
talvasconcelos merged 11 commits into
mainfrom
feat/fiat_denominated_lnurlw
Apr 30, 2026
Merged

Feat: add fiat denominated withdraw links #71
talvasconcelos merged 11 commits into
mainfrom
feat/fiat_denominated_lnurlw

Conversation

@talvasconcelos
Copy link
Copy Markdown
Collaborator

@talvasconcelos talvasconcelos commented Mar 2, 2026

Closes #44

@talvasconcelos talvasconcelos force-pushed the feat/fiat_denominated_lnurlw branch from 4f5410f to 70511ea Compare March 31, 2026 10:42
Comment thread views_lnurl.py
@dni
Copy link
Copy Markdown
Member

dni commented Apr 1, 2026

on pay_links, this is solbed by using.

    # database only allows int4 entries for min and max. For fiat currencies,
    # we multiply by data.fiat_base_multiplier (usually 100) to save the value in cents.
    if data.currency and data.fiat_base_multiplier:
        data.min *= data.fiat_base_multiplier
        data.max *= data.fiat_base_multiplier```

Comment thread models.py
@talvasconcelos
Copy link
Copy Markdown
Collaborator Author

on pay_links, this is solbed by using.

    # database only allows int4 entries for min and max. For fiat currencies,
    # we multiply by data.fiat_base_multiplier (usually 100) to save the value in cents.
    if data.currency and data.fiat_base_multiplier:
        data.min *= data.fiat_base_multiplier
        data.max *= data.fiat_base_multiplier```

there's no fiat multiplier here!

@talvasconcelos
Copy link
Copy Markdown
Collaborator Author

can I merge this one @dni ?

@talvasconcelos talvasconcelos merged commit d90e9d9 into main Apr 30, 2026
6 checks passed
@talvasconcelos talvasconcelos deleted the feat/fiat_denominated_lnurlw branch April 30, 2026 12:08
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.

Selectable currency when creating LNURLw links

2 participants