Skip to content

Conversation

@TrevorNeedham
Copy link
Contributor

For now, I've just duplicated things over to the teacher app, removing student specific elements. I'll also create a tech debt ticket to help us remember to take another look at our common mobius code and see what should be pulled out into a new or existing module.

@CLAassistant
Copy link

CLAassistant commented Apr 11, 2019

CLA assistant check
All committers have signed the CLA.

@TrevorNeedham TrevorNeedham merged commit f0df8a0 into master Apr 15, 2019
@TrevorNeedham TrevorNeedham deleted the teacher/MBL-12224-AddMobiusToTeacher branch April 15, 2019 14:49
kristofnemere added a commit that referenced this pull request Nov 24, 2025
Fixed multiple issues where ToDo list didn't automatically refresh
after changes in various parts of the app:

Changes:
- ToDoListViewModel: Added CalendarSharedEvents listener to refresh
  on RefreshToDoList events
- EditDashboardViewModel: Emit RefreshToDoList when favoriting/
  unfavoriting courses to update filtered ToDo items
- SubmissionWorker: Emit RefreshToDoList after successful assignment
  submissions to remove completed items

This fixes the following QA findings from MBL-19553:
- Issue #2: Badge updates but page doesn't refresh after creating
  events/todos
- Issue #4: Favorite courses filter doesn't auto-update
- Issue #5: Assignment submission doesn't refresh ToDo page
- Issue #9: ToDo created from widget appears on widget but not page
- Issue #10: Deleted event/ToDo doesn't disappear from page

Test plan:
- Create/update/delete calendar events and verify ToDo list refreshes
- Favorite/unfavorite courses and verify filtered ToDos update
- Submit assignments and verify they disappear from ToDo list
- Create ToDos from widget and verify they appear in list
- Delete events/ToDos and verify they disappear immediately

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants