Skip to content

increase tests trade range#1937

Merged
KoalaSat merged 2 commits intoRoboSats:mainfrom
jerryfletcher21:increase-test-trade-range
May 20, 2025
Merged

increase tests trade range#1937
KoalaSat merged 2 commits intoRoboSats:mainfrom
jerryfletcher21:increase-test-trade-range

Conversation

@jerryfletcher21
Copy link
Contributor

What does this PR do?

21 usd as the minimum range used for the tests has fallen below the MIN_ORDER_SIZE that is 20000 sats. Running the tests resulted in errors.

Checklist before merging

  • Install pre-commit and initialize it: pip install pre-commit, then pre-commit install. Pre-commit installs git hooks that automatically check the codebase. If pre-commit fails when you commit your changes, please fix the problems it points out.

@KoalaSat
Copy link
Member

Ey @jerryfletcher21 thanks for all these PRs, really glad to see some help on maintenance. Please accept a small tip for your work. Post here a LN invoice with long expiration date for 65000 satoshis.

@KoalaSat KoalaSat merged commit 7b6cfb8 into RoboSats:main May 20, 2025
1 check passed
@jerryfletcher21
Copy link
Contributor Author

jerryfletcher21 commented May 20, 2025

@KoalaSat thank you!

lnbc650u1p5zefrdsp5zcqzg23zs5uvrdcwyxux79q7090qwqws458rxm55junmmumyrq0spp5ff7f8x2jlvephr63tsedgh4mrh38mc7x4mxgcrlx0n0xvjvjae8sdqqxq9yn4qqcqzysrzjqdllz2m2fe9ukju5fdkjptcgehlkrv6xrswl7rgq4zrfws2d3yduwr8j0yqpv2sqqqqqqqlgqqqqp9sq2q9qxpqysgq8sfsvtptxu8hjp49vvnx6dy7w3dv0gcq9nw24kldepljz2dl58crj4s9xhtjjfcttwexpf9phy92da7cx0vgf7j844zd8lmcrdpttzgqpvvwgr

@KoalaSat
Copy link
Member

KoalaSat commented May 21, 2025

@KoalaSat thank you!

lnbc650u1p5zefrdsp5zcqzg23zs5uvrdcwyxux79q7090qwqws458rxm55junmmumyrq0spp5ff7f8x2jlvephr63tsedgh4mrh38mc7x4mxgcrlx0n0xvjvjae8sdqqxq9yn4qqcqzysrzjqdllz2m2fe9ukju5fdkjptcgehlkrv6xrswl7rgq4zrfws2d3yduwr8j0yqpv2sqqqqqqqlgqqqqp9sq2q9qxpqysgq8sfsvtptxu8hjp49vvnx6dy7w3dv0gcq9nw24kldepljz2dl58crj4s9xhtjjfcttwexpf9phy92da7cx0vgf7j844zd8lmcrdpttzgqpvvwgr

4e19483a264f89c5ce11ee8c416982b75e13de47b24e334779b4526e07387861

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.

2 participants