Skip to content

Create Team and TeamMembership models - #811

Open
AdrianDAlessandro wants to merge 2 commits into
mainfrom
569-add-teams-models
Open

AdrianDAlessandro wants to merge 2 commits into
mainfrom
569-add-teams-models

Conversation

@AdrianDAlessandro

Copy link
Copy Markdown
Collaborator

Description

This PR creates the Team and Team Membership models required to enable Teams features.

Each Team can have any number of members and any number of those members can be a manager.

None of this is visible to users yet, other than admin users that can see these in the database.

Fixes #569

Type of change

  • Documentation (non-breaking change that adds or improves the documentation)
  • New feature (non-breaking change which adds functionality)
  • Optimization (non-breaking, back-end change that speeds up the code)
  • Technical work (non-breaking, change which is work as part of a new feature)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Key checklist

  • All tests pass (eg. python -m pytest)
  • The documentation builds and looks OK (eg. mkdocs serve)
  • Pre-commit hooks run successfully (eg. pre-commit run --all-files)

Further checks

  • Code is commented, particularly in hard-to-understand areas
  • Tests added or an issue has been opened to tackle that in the future. (Indicate issue here: # (issue))

@codecov

codecov Bot commented Oct 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.66667% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
main/models/team_models.py 91.66% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@Sahil590 Sahil590 left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

From what we discussed earlier
LGTM!

This branch has not been deployed

No deployments
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.

Add models for teams of users

3 participants