Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: introduce convenience Status::verify_is_active method #1005

Merged
merged 3 commits into from
Dec 13, 2023
Merged

feat: introduce convenience Status::verify_is_active method #1005

merged 3 commits into from
Dec 13, 2023

Commits on Dec 12, 2023

  1. feat: introduce convenience Status::verify_active method 

    Similarly to ChannelEnd::verify_not_closed, introduce a convenience
method Status::verify_active which checks whether client’s status is
active and returns an error if it isn’t.  This simplifies whole bunch
of code locations making them more DRY.
    @mina86
    mina86 committed Dec 12, 2023
    Configuration menu
    Copy the full SHA
    99020c2 View commit details
    Browse the repository at this point in the history

Commits on Dec 13, 2023

  1. Update .changelog/unreleased/improvements/1005-status-verify-active.md 

    Co-authored-by: Farhad Shabani <[email protected]>
Signed-off-by: Michal Nazarewicz <[email protected]>
    @mina86 @Farhad-Shabani
    mina86 and Farhad-Shabani authored Dec 13, 2023
    Configuration menu
    Copy the full SHA
    ad7f68b View commit details
    Browse the repository at this point in the history

  2. review

    @mina86
    mina86 committed Dec 13, 2023
    Configuration menu
    Copy the full SHA
    b1f7a1e View commit details
    Browse the repository at this point in the history