Remove overloading of /api/v1/teams endpoint

[knolleary]

Description

The /api/v1/teams endpoint served two purposes:

• If admin, return a paginated list of teams on the platform
• If ?slug=<slug> is provided, provide details of the corresponding team - with the appropriate ACL checks

This overloading is not nice when it comes to documenting the API. This PR cleans things up by introducing a separate endpoint to get a team by its slug: /api/v1/teams/slug/:teamSlug.

Added unit test coverage of the new endpoint; the previous version was not explicitly covered.

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 flowforge/helm to update ConfigMap Template
  • Issue/PR raised on flowforge/CloudProject to update values for Staging/Production

Labels

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

[Pezmc]

Changes look good, I can't explain why that UI test failed, but I've restarted it.

[knolleary]

The UI test failure can be recreated locally on this branch, but not on main. So this has definitely done something. I'm digging.