Skip to content

Offline Mode #674

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.

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

Closed
wants to merge 35 commits into from
Closed

Offline Mode #674

wants to merge 35 commits into from

Conversation

wk9874
Copy link
Collaborator

@wk9874 wk9874 commented Feb 3, 2025

Add Offline Mode

Python Version(s) Tested: 3.10

Operating System(s): Ubuntu

Documentation PR: Issue on Docs repo.

📝 Summary

Added Offline mode sender

🔄 Changes

Various

✔️ Checklist

  • Unit and integration tests passing.
  • Pre-commit hooks passing.
  • Quality checks passing.
  • Updated the documentation.

@wk9874 wk9874 changed the title Wk9874/uploader Offline Mode Feb 3, 2025
@kzscisoft kzscisoft added the enhancement New feature or request label Feb 3, 2025
@kzscisoft kzscisoft self-assigned this Feb 3, 2025
@kzscisoft kzscisoft added this to the Python API v2 milestone Feb 3, 2025
@kzscisoft kzscisoft linked an issue Feb 3, 2025 that may be closed by this pull request
2 tasks
@@ -58,6 +70,9 @@ def get(
identifier for an alert
the alert itself as a class instance
"""
if offline:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should this raise an Exception?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I dont think so - we dont want it to stop executing, just to use the currently staged changes instead of fetching

@kzscisoft kzscisoft closed this Feb 10, 2025
@kzscisoft kzscisoft deleted the wk9874/uploader branch March 9, 2025 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Simvue Server v3
2 participants