-
Notifications
You must be signed in to change notification settings - Fork 0
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
Communication
: Save failed-to-send message
#84
Communication
: Save failed-to-send message
#84
Conversation
…com:ls1intum/artemis-ios into feature/communication/retry-offline-message
ContextProblemNetworks are sometimes constrained or expensive. MotivationUsers want to go on messaging. Requirements EngineeringExisting (Problematic) Solution / SystemThe app starts loading, if the user tapped the send button. Proposed SystemThe send-message request starts loading, if the user tapped the send button. SuccessThe message is inserted into the appropriate section. FailureThe message remains in the queue. The queue is persisted across app launches/life cycles. (The user can remove the message.) RequirementsFunctional Requirements
Non-functional Requirements
AnalysisAnalysis Object Model
Dynamic Behavior. |
- Add offline messages - Redact author
Simulator.Screen.Recording.-.iPhone.15.Pro.Max.-.2024-03-09.at.19.09.19.mp4 |
Thread 14: "-[UIKeyboardTaskQueue waitUntilAllTasksAreFinished] may only be called from the main thread."
Simulator.Screen.Recording.-.iPhone.15.Pro.Max.-.2024-03-10.at.22.28.42.mp4Simulator.Screen.Recording.-.iPhone.15.Pro.Max.-.2024-03-10.at.22.30.16.mp4 |
RevisionRetry manually only after sending failed.
|
- Redundant Optional Initialization Violation: Initializing an optional variable with nil is redundant (redundant_optional_initialization)
This reverts commit 6c86d8c.
Warning
Database is in-memory only
Part #48
Android