imchristie
imchristie
@seanlip PTAL
@seanlip PTAL
@seanlip The merge conflict is fixed now. I am still trying to figure out how to make that function private after changing the file logged-in-user-utils to a typescript file. So...
@seanlip Yes that sounds good to me. The "private function" thing is taking more time than I thought.
@agallop, seem like there's a failure in CI for logged-out-user/check-all-user-flow-of-partner because you removed the function navigateToAboutFoundationPageViaFooter.
@agallop @seanlip PTAL
@seanlip PTAL. @Akhilesh-max Take a look if you have time.
Hi @HardikGoyal2003, after discussing with the acceptance tests team, we have agreed on merging this PR after June 28 to avoid causing a big merge conflict for the contributors in...
@amyyeung17 The new acceptance test(manage-classroom-progress-in-home-learner-dashboard) that you added in this PR is failing in CI. Please have a look at it before attempting to merge.
@souja450 Just a quick note, there's a lot of checks failing in CI. Please fix it to make sure the PR passes all the CI checks.