XRPLF / xrpl-py

A Python library to interact with the XRP Ledger (XRPL) blockchain
ISC License
150 stars 84 forks source link

Update faucet_url processing logic #690

Closed ckeshava closed 6 months ago

ckeshava commented 7 months ago

High Level Overview of Change

This PR seeks to address https://github.com/XRPLF/xrpl-py/issues/364

This is an alternative approach competing with https://github.com/XRPLF/xrpl-py/pull/676 PR #676 needs to be closed after this PR is reviewed and merged.

Context of Change

Type of Change

Did you update CHANGELOG.md?

Test Plan

Some unit tests have been added to validate this change.

Many thanks to @justinr1234 for coming up with the design and code for this solution

jonathanlei commented 6 months ago

LGTM