fix: Handle session timeout during Quiz Creation in Coach

[GarvitSinghal47]

Fixes #11773

Summary

1. Modified the router.beforeEach guard in Coach to handle session timeouts.
2. Added a check for user login status.
3. If the session times out, redirected users to the 'AuthMessage' page with the 'next' query parameter set to the path they were trying to access.
4. After signing in, users are redirected back to the original destination.

…

References

…

Reviewer guidance

…

---

Testing checklist

• Contributor has fully tested the PR manually
• If there are any front-end changes, before/after screenshots are included
• Critical user journeys are covered by Gherkin stories
• Critical and brittle code paths are covered by unit tests

PR process

• PR has the correct target branch and milestone
• PR has 'needs review' or 'work-in-progress' label
• If PR is ready for review, a reviewer has been added. (Don't use 'Assignees')
• If this is an important user-facing change, PR or related issue has a 'changelog' label
• If this includes an internal dependency change, a link to the diff is provided

Reviewer checklist

• Automated test coverage is satisfactory
• PR is fully functional
• PR has been tested for accessibility regressions
• External dependency files were updated if necessary (yarn and pip)
• Documentation is updated
• Contributor is in AUTHORS.md

[github-actions]

Build Artifacts

Asset type	Download link
PEX file	kolibri-.pex
Windows Installer (EXE)	kolibri-0.16.1.dev0+git.20240414101445-unsigned.exe
Debian Package	kolibri_0.16.1.dev0+git.20240414101445-0ubuntu1_all.deb
Mac Installer (DMG)	kolibri-0.16.1.dev0+git.20240414101445-0.4.0.dmg
Android Package (APK)	kolibri-0.16.1.dev0+git.20240414101445-0.1.0-debug.apk
TAR file	kolibri-0.16.1.dev0+git.20240414101445.tar.gz
WHL file	kolibri-0.16.1.dev0+git.20240414101445-py2.py3-none-any.whl

[nucleogenesis]

Thank you @GarvitSinghal47 - this is a much better user experience when timing out.

[AllanOXDi]

I am not well-versed- still learning about this. @rtibbles can you help and give a final look and we it merged?

[rtibbles]

There are a couple of issues with the current approach, and I think it might be neater to use the existing permissions tracking components for this.

[rtibbles]

Hrm, it might be worth investigating the withAuthMessage component defined here:

kolibri/kolibri/plugins/device/assets/src/views/withAuthMessage.js

Line 17 in 4dae628

export default function withAuthMessage(component, authorizedRole) {

(search the code base for other uses), alternatively, the AuthMessage component can be used inside the Coach index page.

This particular implementation will kind of work, although I am fairly sure the next query parameter will be lost during the redirect to the auth page.

It is also possible for Kolibri to be configured to redirect unauthenticated users to the Learn page, so in that case, this redirect will not prompt someone to login.

[nucleogenesis]

@GarvitSinghal47 -- would you be able to revise this PR according to the feedback above?

[GarvitSinghal47]

@nucleogenesis , my apologies for the oversight. I completely forgot about this open pull request. I will make the necessary adjustments and provide the updates promptly.

[GarvitSinghal47]

@rtibbles, I attempted something and am simply seeking clarification: Is the video below the desired output we're aiming for?

User.Sign.In.-.Kolibri.mp4

[rtibbles]

Yes, that looks like the same sort of pattern as we have used elsewhere.

[nucleogenesis]

@GarvitSinghal47 - did something get lost in a merge or something? Seems there is only a single small change here.