Skip to content

Amount ser/deser#4699

Draft
modship wants to merge 10 commits intomainfrom
api_amount
Draft

Amount ser/deser#4699
modship wants to merge 10 commits intomainfrom
api_amount

Conversation

@modship
Copy link
Member

@modship modship commented May 31, 2024

  • document all added functions
  • try in sandbox /simulation/labnet
    • if part of node-launch, checked using the resync_check flag
  • unit tests on the added/changed features
    • make tests compile
    • make tests pass
  • add logs allowing easy debugging in case the changes caused problems
  • if the API has changed, update the API specification
@modship modship linked an issue Jun 3, 2024 that may be closed by this pull request
Improve Amount usage in api #4688
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@modship