Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implementing multi tenancy #345

Merged
merged 46 commits into from
May 23, 2024
Merged

Implementing multi tenancy #345

merged 46 commits into from
May 23, 2024

Conversation

DennisAlund
Copy link
Member

@DennisAlund DennisAlund commented May 21, 2024

Description

Implements multi tenancy by adding route parameter http://localhost:3000/{site} which corresponds to a document in the database tanam/{site}

Referenced issues

Closes #342

Screen recording

May-22-2024 15-51-48

@DennisAlund DennisAlund self-assigned this May 21, 2024
@DennisAlund DennisAlund changed the base branch from main to next May 21, 2024 15:02
hosting/src/queries/site.ts Outdated Show resolved Hide resolved
hosting/src/models/tanamSite.ts Show resolved Hide resolved
hosting/src/models/tanamSite.ts Outdated Show resolved Hide resolved
@DennisAlund DennisAlund added this to the google-io-ext milestone May 22, 2024
Copy link

github-actions bot commented May 22, 2024

Visit the preview URL for this PR (updated for commit f8e84f6):

https://tanam-testing--pr345-issue-342-multi-tena-rjisbbts.web.app

(expires Thu, 30 May 2024 03:05:28 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 5bbe8a1a68d9684d656bffba10008fe532204561

@DennisAlund DennisAlund marked this pull request as ready for review May 22, 2024 07:46
@DennisAlund
Copy link
Member Author

There is a known issue with this one: there is no implementation for good handling of bad route parameter. See #347

@muzanella11 muzanella11 self-requested a review May 22, 2024 16:39
hosting/.prettierrc Outdated Show resolved Hide resolved
@DennisAlund DennisAlund merged commit 8441a19 into next May 23, 2024
3 checks passed
@DennisAlund DennisAlund deleted the issue/342-multi-tenancy branch May 23, 2024 03:08
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.

Implement Multi-Tenancy for Tanam CMS
3 participants