Skip to content
This repository has been archived by the owner on Dec 12, 2024. It is now read-only.

Add type-safe layer and filtering #23

Merged
merged 9 commits into from
Jul 5, 2024

Conversation

aparkersquare
Copy link
Contributor

@aparkersquare aparkersquare commented Jul 3, 2024

  • Add TypedMoneyAddress, Currency and Protocol classes, with values from the DAP spec
  • Add BtcOnChainAddress, BtcLightningAddress, MobileMoneyAddress
  • Add TypedMoneyAddressesExample
  • Add some more doc-strings

Valuable feedback from @cwmyers helped shape the design of this type-safe layer. Thanks Chris!

Comment on lines 9 to 10
* - a `handle` which is a unique identifier for the user within the domain
* - the `domain` of the DAP registry.
Copy link

Choose a reason for hiding this comment

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

Consider using @Property for the properties, as shown here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the pointer, I've updated to use @Property.

@aparkersquare aparkersquare force-pushed the aparker/20240628-typesafe-filtering-layer branch from d131187 to 803eaca Compare July 3, 2024 12:03
Copy link

@cwmyers cwmyers left a comment

Choose a reason for hiding this comment

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

Love it! I'm still in two minds about the registry concept but it has prior art in the dependent APIs so I'm cool with it.

@aparkersquare aparkersquare merged commit 0062324 into main Jul 5, 2024
2 checks passed
@aparkersquare aparkersquare deleted the aparker/20240628-typesafe-filtering-layer branch July 5, 2024 01:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants