Classes Xapi - #265
Open
Mandvii wants to merge 155 commits into
Open
Conversation
…idate student existence before group creation in `ClazzDetailViewModel`.
…nd map them to XapiAgent objects on save.
…ntGroupingEditViewModel`.
…Model`. * Update `StudentGroupingEditScreen` to handle group name changes and display validation errors. * Extend `Clazz` model and database entities to support a `metadata` field for storing group IDs. * Implement `onClickSave` in `StudentGroupingEditViewModel` to generate xAPI statements for group creation and update class metadata. * Add `VERB_CREATED` and `VERB_UPDATED` constants to `XapiVerb`. * Update navigation and UI padding for the student grouping entry point.
…StudentGroupingDetailViewModel UI state.
* Add `getGroupDetail` and `getGroupsByIds` to `XapiActorDataSource` and its database implementation. * Update `ClazzDetailViewModel` to extract and load group details from class metadata. * Enhance `ClazzDetailScreen` to display a list of student groups with toggle functionality. * Implement member listing in `StudentGroupingDetailScreen` and populate it via `StudentGroupingDetailViewModel`. * Fix route parameter naming from `guid` to `classUid` in `StudentGroupingEdit` and related logic. * Remove unused database schema file.
* Add `getGroupDetail` and `getGroupsByIds` to `XapiActorDataSource` and its database implementation. * Update `ClazzDetailViewModel` to extract and load group details from class metadata. * Enhance `ClazzDetailScreen` to display a list of student groups with toggle functionality. * Implement member listing in `StudentGroupingDetailScreen` and populate it via `StudentGroupingDetailViewModel`. * Fix route parameter naming from `guid` to `classUid` in `StudentGroupingEdit` and related logic. * Remove unused database schema file.
…t grouping screens.
…t grouping screens.
…redundant navigation steps in subflows.
… class name from "New Class" to "TestClass" across all steps and assertions.
* Added a delete confirmation dialog to `StudentGroupingDetailScreen` using `RespectBasicAlertDialog`. * Implemented `onClickDeleteGroup`, `onDismissDeleteGroupDialog`, and `onConfirmDeleteGroup` logic in `StudentGroupingDetailViewModel`. * Integrated xAPI statement generation for the "deleted" verb when a group is removed. * Updated `StudentGroupingEditViewModel` to correctly use the "updated" xAPI verb when modifying existing groups. * Added `VERB_DELETED` constant and new string resources for deletion UI.
…oss various ViewModels.
…oss various ViewModels.
…oss various ViewModels.
…s to the flows directory.
…s to the flows directory.
…t.yaml` to include navigation to classes, remove redundant wait timers, and add assertions for student assignment results.
…t.yaml` to remove redundant navigation to the Classes tab and assertions for "TestClass" during student verification steps.
…t.yaml` to change the target class name from "TestGroup" to "TestClass".
…date related logic. - Refactor `AssignmentEditScreen` and `AssignmentEditViewModel` to use `Clazz` options instead of `XapiGroup` for the assignee field. - Update `UpdateClazzStudentXapiGroupUseCase` to resolve class names via xAPI activity statements rather than direct class lookup. - Add debug logging to `ClazzDetailViewModel` and `UpdateClazzStudentXapiGroupUseCase` to track group member synchronization. - Map `Clazz` objects to xAPI groups using `studentsXapiGroup` extension when selecting an assignee.
…tion and remove debug logs in `ClazzDetailViewModel`.
…se it in `UpdateClazzStudentXapiGroupUseCase`.
…se it in `UpdateClazzStudentXapiGroupUseCase`.
…zzStudentXapiGroupUseCase`.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.