chore: Remove live update for study group dashboard#3057
chore: Remove live update for study group dashboard#3057arkirchner wants to merge 1 commit intomainfrom
Conversation
Live updates have been disabled for 5 years. Related to #b2fc574
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3057 +/- ##
==========================================
+ Coverage 70.08% 70.25% +0.16%
==========================================
Files 215 213 -2
Lines 6850 6814 -36
==========================================
- Hits 4801 4787 -14
+ Misses 2049 2027 -22 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I am sorry. I realized an issue. The WebSocket code is rendering the initial UI even if no messages are received. This needs to be moved out to visualize the user progress. |
|
Thanks for working on this part and opening a PR early. I would kindly ask not to remove the dashboard and/or the live update features. For sure, they have been disabled for about five years and were not fully functional. Resolving these issues has been (and still is) an item on my personal follow-up list. There is a local branch with some changes lying around that I would like to finalize soon. Disabling the live updates occurred at the beginning of the COVID-19 pandemic at a time when we just launched a programming course on openHPI. At that time, we had some performance regressions, which ultimately led to a multi-year refactoring of the code execution workflow (and thus is part of our journey towards Poseidon). Due to the difficult times back then and the stress we experienced, we aimed to identify potential issues and disabled components not completely necessary for that course. As I know today, disabling |
Live updates have been disabled for 5 years. Seems safe to remove.
Related to b2fc574