Skip to content

Conversation

@msouff
Copy link
Contributor

@msouff msouff commented Nov 12, 2025

Description

This PR adds Django Tenants as an optional dependency to Tethys Platform. This enables multi-tenancy in the Tethys Portal, meaning that a single portal instance can serve multiple isolated tenants/organizations. Besides facilitating a SaaS architecture, this will allow us to customize Tethys portals, apps, and any other resource based on organizations. E.g. portal style and available apps could vary based on tenant.

image image image

Notice the different styles and admin for each tenant.

Quality Checks

  • At least one new test has been written for new code
  • New code has 100% test coverage
  • Code has been formatted with Black
  • Code has been linted with flake8
  • Docstrings for new methods have been added
  • The documentation has been updated appropriately

Copy link
Contributor

@shawncrawley shawncrawley left a comment

Choose a reason for hiding this comment

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

I have just one question - and of course I see the tests are failing. Overall this will be a very cool feature to have.

@coveralls
Copy link

coveralls commented Nov 13, 2025

Coverage Status

coverage: 100.0%. remained the same
when pulling ee310b0 on Aquaveo:tethys_tenants
into 5f547aa on tethysplatform:main.

Copy link
Contributor

@shawncrawley shawncrawley 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!

@msouff msouff marked this pull request as ready for review November 19, 2025 23:40
Copy link
Member

Choose a reason for hiding this comment

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

What is this file?

@msouff msouff requested a review from swainn December 23, 2025 17:49
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.

6 participants