Fix race condition when changing teams while on the application page

[cstns]

Issue caused by setting team sequentially while on the application page. This did not pose a problem until nesting the application pages under the team namespace

Description

• removed the team dispatch on beforeMount and beforeRouteEnter causing the issue
• update how we change the team, updating the team is more than enough while pushing a new route for browser history consistency
• a bit of code cleanup on the application page

Related Issue(s)

closes #4950
introduced by #4885

Checklist

• I have read the contribution guidelines
• Suitable unit/system level tests have been added and they pass
• Documentation has been updated
  • Upgrade instructions
  • Configuration details
  • Concepts
• Changes flowforge.yml?
  • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
  • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production

Labels

• Includes a DB migration? -> add the area:migration label

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.44%. Comparing base (cf9bfbe) to head (923aab4).
Report is 8 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4951   +/-   ##
=======================================
  Coverage   78.44%   78.44%           
=======================================
  Files         325      325           
  Lines       15353    15353           
  Branches     3529     3529           
=======================================
  Hits        12043    12043           
  Misses       3310     3310           

Flag	Coverage Δ
backend	78.44% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Steve-Mcl]

Should this not simply return [] if team is not set?

As it stands, you will return at array containing '/team/undefined/billing' etc.

[knolleary]

@Steve-Mcl have you tested it? I can confirm this PR fixes the bug that is live in production now that we should get fixed. I don't see any ill effects because of this.

[Steve-Mcl]

have you tested it?

Yes, it works well, however I only flagged this code section because the PR (on the whole) is about handling team being undefined. By the same virtue, if team IS undefined, then we will get a bunch of links to /undefined/

I am not blocking it - happy for the PR to be merged as is.