-
Notifications
You must be signed in to change notification settings - Fork 108
[MBL-19126][Student][Parent] Assignment Details DCP #3259
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
Conversation
kdeakinstructure
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
QA 👍
Please merge the master and resolve conflicts before merge.
I saw you put 'Stub' on these 2 tests:
testComments_addCommentToSingleAttemptSubmission
testComments_addCommentToMultipleAttemptSubmission
but these are already fixed on master so make sure before you merge that this won't be stubbed unintentionally.
kristofnemere
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not working offline in Student, I'll fix that later just wanted to block merging
# Conflicts: # apps/parent/src/androidTest/java/com/instructure/parentapp/utils/ParentComposeTest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/e2e/classic/FilesE2ETest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/e2e/classic/GradesE2ETest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/e2e/classic/ModulesE2ETest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/e2e/classic/ShareExtensionE2ETest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/e2e/classic/k5/ImportantDatesE2ETest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/e2e/classic/k5/ScheduleE2ETest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/e2e/classic/offline/OfflineGradesE2ETest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/e2e/classic/offline/OfflineModulesE2ETest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/interaction/ModuleInteractionTest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/interaction/NotificationInteractionTest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/interaction/PickerSubmissionUploadInteractionTest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/interaction/ScheduleInteractionTest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/interaction/SubmissionDetailsInteractionTest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/interaction/TodoInteractionTest.kt # apps/student/src/androidTest/java/com/instructure/student/ui/utils/StudentTest.kt
…' into MBL-19126-assignment-details-dcp # Conflicts: # apps/student/src/androidTest/java/com/instructure/student/ui/utils/StudentComposeTest.kt
📊 Code Coverage Report✅ Student
✅ Teacher
|
libs/pandautils/schemas/com.instructure.pandautils.room.appdatabase.AppDatabase/12.json
Outdated
Show resolved
Hide resolved
# Conflicts: # libs/pandautils/src/test/java/com/instructure/pandautils/features/assignments/details/AssignmentDetailsViewModelTest.kt
|
refs: MBL-19126 affects: Student, Teacher, Parent release note:
This reverts commit c036045.
Test plan:
Create a discussion checkpoint assignment
Go to assignment details screen in Student and Parent app
refs: MBL-19126
affects: Student, Parent
release note: Details page of assignments with discussion checkpoints now shown with proper data
Checklist