Skip to content
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

Mobile 4028 #4245

Merged
merged 4 commits into from
Nov 25, 2024
Merged

Mobile 4028 #4245

merged 4 commits into from
Nov 25, 2024

Conversation

dpalou
Copy link
Contributor

@dpalou dpalou commented Nov 21, 2024

No description provided.

@dpalou dpalou force-pushed the MOBILE-4028 branch 4 times, most recently from ea9b49c to d27a958 Compare November 25, 2024 08:36
src/core/services/sites.ts Outdated Show resolved Hide resolved
src/core/services/sites.ts Show resolved Hide resolved
Now it uses a logout page so Angular guards are triggered before doing the logout process.
Before MOBILE-4028, links could be treated before authenticating to a site, and this meant that the 'navigateToModule' function used 'getAndOpenCourse', which opened first the course and then the activity. Now the links are treated after authenticating, and this means that the activity is opened directly, without opening the course first.
@crazyserver crazyserver merged commit 476a179 into moodlehq:main Nov 25, 2024
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants