supervisor icon indicating copy to clipboard operation
supervisor copied to clipboard

Simplify backup restore failure messaging and add call-to-action

Open codyc1515 opened this issue 1 year ago • 1 comments

Proposed change

Simplify backup restore failure messaging and add call-to-action.

Type of change

  • [ ] Dependency upgrade
  • [ ] Bugfix (non-breaking change which fixes an issue)
  • [ ] New feature (which adds functionality to the supervisor)
  • [ ] Breaking change (fix/feature causing existing functionality to break)
  • [x] Code quality improvements to existing code or addition of tests

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue:
  • Link to documentation pull request:
  • Link to cli pull request:

Checklist

  • [ ] The code change is tested and works locally.
  • [x] Local tests pass. Your PR cannot be merged unless tests pass
  • [x] There is no commented out code in this PR.
  • [x] I have followed the [development checklist][dev-checklist]
  • [x] The code has been formatted using Black (black --fast supervisor tests)
  • [ ] Tests have been added to verify that the new code works.

codyc1515 avatar Jan 28 '24 03:01 codyc1515

@codyc1515 there is a test failure in the test_restore_immediate_errors test case. The test case checks the message. Can you update the test case as well? Thanks!

agners avatar Feb 14 '24 08:02 agners

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.

github-actions[bot] avatar Mar 16 '24 22:03 github-actions[bot]