Skip to content

Commit 04eca3b

Browse files
committed
Updates snackbars, fixes messages when bulk actions complete
1 parent f8f8860 commit 04eca3b

File tree

3 files changed

+22
-25
lines changed

3 files changed

+22
-25
lines changed

kolibri/plugins/facility/assets/src/views/users/UsersRootPage/index.vue

Lines changed: 10 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -303,7 +303,9 @@
303303
}
304304
305305
function navigateToSidePanel(sidePanelName) {
306-
const newRoute = overrideRoute(route, { name: sidePanelName });
306+
const { query } = route;
307+
delete query.failedActionType;
308+
const newRoute = overrideRoute(route, { name: sidePanelName, query });
307309
router.push(newRoute);
308310
}
309311
@@ -358,15 +360,12 @@
358360
return {};
359361
});
360362
361-
362-
// Alerting users about enrollment/assignment errors
363-
const alertDismissed = ref(false)
364-
365-
function handleAlertDismissal() {
366-
alertDismissed.value = true
367-
}
368-
369-
const showingInvalidUsers = computed(() => !alertDismissed.value && Boolean(route.query.by_ids));
363+
const showingInvalidUsers = computed(() => {
364+
return (
365+
Boolean(route.query.by_ids) &&
366+
Boolean(route.query.failedActionType)
367+
);
368+
});
370369
const warningMessage = computed(
371370
() => {
372371
switch(route.query?.failedActionType) {
@@ -375,13 +374,13 @@
375374
case(InvalidActionTypes.ENROLL):
376375
return someFailedToEnroll$();
377376
default:
377+
// When the page loads
378378
return '';
379379
}
380380
}
381381
);
382382
383383
return {
384-
handleAlertDismissal,
385384
warningMessage,
386385
showingInvalidUsers,
387386
windowIsSmall,
@@ -526,12 +525,6 @@
526525
}
527526
},
528527
},
529-
beforeRouteLeave(to,__,next) {
530-
this.alertDismissed = false;
531-
const { query } = to;
532-
delete query.failedActionType;
533-
next(this.overrideRoute(to, { query }))
534-
}
535528
};
536529
537530
</script>

kolibri/plugins/facility/assets/src/views/users/sidePanels/AssignCoachesSidePanel.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -319,7 +319,7 @@
319319
}
320320
}
321321
if(invalidRoles.value?.length) {
322-
return unableToAssignAlert$();
322+
return someFailedToAssign$();
323323
}
324324
};
325325

kolibri/plugins/facility/assets/src/views/users/sidePanels/EnrollLearnersSidePanel.vue

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -137,14 +137,17 @@
137137
CloseConfirmationGuard,
138138
},
139139
mixins: [commonCoreStrings],
140-
setup(props) {
140+
setup(props, { emit }) {
141141
const store = getCurrentInstance().proxy.$store;
142142
const route = useRoute();
143143
const router = useRouter();
144144
const goBack = useGoBack({
145145
getFallbackRoute: () => {
146+
const { query } = route;
147+
delete query.failedActionType;
146148
return overrideRoute(route, {
147149
name: getRootRouteName(route),
150+
query,
148151
});
149152
},
150153
});
@@ -220,7 +223,6 @@
220223
async function _enrollLearners() {
221224
loading.value = true;
222225
const enrollments = selectedOptions.value.flatMap(collection_id => {
223-
const alreadyEnrolled = classMembershipsByUser.value;
224226
return Array.from(props.selectedUsers)
225227
.map(user => ({ collection: collection_id, user }));
226228
});
@@ -239,9 +241,10 @@
239241
240242
// nextTick to give the hasUnsavedChanges a chance to react
241243
// so that the confirmation guard doesn't trigger in this case
242-
nextTick(() => {
243-
if(invalid?.length) {
244-
return router.push(
244+
return nextTick(() => {
245+
// This doesn't need to be done in the assign side panel but IDK for sure why
246+
if(Boolean(invalid)) {
247+
router.push(
245248
overrideRoute(
246249
route,
247250
{
@@ -253,8 +256,9 @@
253256
},
254257
)
255258
);
259+
emit('clearSelection');
260+
return true;
256261
} else {
257-
goBack();
258262
return true;
259263
}
260264
});
@@ -285,7 +289,7 @@
285289
return usersEnrolledNotice$();
286290
}
287291
}
288-
if(invalidRoles.value?.length) {
292+
if(invalidMemberships.value?.length) {
289293
return someFailedToEnroll$();
290294
}
291295
};

0 commit comments

Comments
 (0)