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

Simplify persistence design for temporarily offline status #9855

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Simplify persistence design for temporarily offline status #9855

wants to merge 5 commits into from
+73 −74

Commits on Oct 11, 2024

  1. Simplify persistence design for temporarily offline status 

    There were complex code to synchronize transient state of Computer from Node. But since any temporarily offline cause is being set by a user and should be persisted in a node, the Computer should actually delegate to Node instead of trying to synchronize it
    @Vlatombe
    Vlatombe committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    187a823 View commit details
    Browse the repository at this point in the history

  2. Fix javadoc

    @Vlatombe
    Vlatombe committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    962a286 View commit details
    Browse the repository at this point in the history

  3. Fix tests and apply suggestions

    @Vlatombe
    Vlatombe committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    888103c View commit details
    Browse the repository at this point in the history

  4. Fix getNodeOrDie()

    @Vlatombe
    Vlatombe committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    0d129d9 View commit details
    Browse the repository at this point in the history

  5. Reduce visibility of Node#isTemporarilyOffline

    @Vlatombe
    Vlatombe committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    1d20b1c View commit details
    Browse the repository at this point in the history