Skip to content

Conversation

@FelixTeutsch
Copy link
Contributor

Changes:

  • App Tour implemented for Students, Supervisors and Admins.
  • Navigate to App Tour after Tags have been selected and arranged
  • Fix UX complain that student adds field is not always visible.
    Screenshot 2025-06-18 at 00 13 19

What is missing: The admin does not get shown the app tour. This has to do with routing, and is beyond me to fix. Go to 01.routing.global.ts and edit the following lines to redirect to app tour if first sign on:

  if (userRole === UserRoles.ADMIN) {
    if (!to.path.startsWith('/admin')) {
      return navigateTo('/admin/dashboard')
    }
    return
  }

@FelixTeutsch FelixTeutsch requested review from LionRise, jb-cc and nb159 June 17, 2025 22:21
@FelixTeutsch FelixTeutsch self-assigned this Jun 17, 2025
…ew translations; add supervisor app tour page
…, including admin, app tour, and profile sections
…ashboard sections; enhance video descriptions for supervisors
…pervisor roles; include new video content and enhance translations
…rvisor roles; update alt text and descriptions for improved clarity
…ashboard sections; enhance descriptions and alt text for improved user experience
…ising title in SupervisorStudentList and currently-supervising components
…cess for students and supervisors to direct to app tour
@FelixTeutsch FelixTeutsch force-pushed the SCRUM-290-Frontend-Implement-App-Tour branch from 2732c03 to bd13329 Compare June 17, 2025 22:37
@FelixTeutsch
Copy link
Contributor Author

FML

@FelixTeutsch FelixTeutsch changed the base branch from main to dev June 17, 2025 22:49
@FelixTeutsch
Copy link
Contributor Author

Un-FML, all good

nb159
nb159 previously approved these changes Jun 18, 2025
Copy link
Contributor

@nb159 nb159 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, good job

Copy link
Contributor

@jb-cc jb-cc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

veni, vidi

@jb-cc jb-cc merged commit 29fd24f into dev Jun 18, 2025
2 checks passed
@jb-cc jb-cc deleted the SCRUM-290-Frontend-Implement-App-Tour branch June 18, 2025 00:57
jb-cc added a commit that referenced this pull request Jun 18, 2025
Co-authored-by: jb-cc <115902941+jb-cc@users.noreply.github.com>
jb-cc added a commit that referenced this pull request Jun 18, 2025
Co-authored-by: jb-cc <115902941+jb-cc@users.noreply.github.com>
nb159 pushed a commit that referenced this pull request Jun 19, 2025
Co-authored-by: jb-cc <115902941+jb-cc@users.noreply.github.com>
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.

4 participants