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

[Multi-Tenant] Tenanted Quotes #3118

Open
4 tasks
Tracked by #2893
njlie opened this issue Nov 21, 2024 · 0 comments · May be fixed by #3171
Open
4 tasks
Tracked by #2893

[Multi-Tenant] Tenanted Quotes #3118

njlie opened this issue Nov 21, 2024 · 0 comments · May be fixed by #3171
Assignees

Comments

@njlie
Copy link
Contributor

njlie commented Nov 21, 2024

Quotes should become tenanted as per the Multi-Tenancy design, and will be logically separated in the backend database by a tenantId row that serves as a foreign key on the tenants table.

  • Add tenantId row to quotes table that is required and maps to an id on the tenants table.
  • Add tenantId to database queries on quotes
  • Acquire tenantId from request headers on routes involving quotes and pass them into quote service logic.
  • Include tenantId in path of quote URL. e.g. http://exampe-rafiki/:tenantId/quotes/:id

Prerequisites

@github-project-automation github-project-automation bot moved this to Backlog in Rafiki Nov 21, 2024
@njlie njlie moved this from Backlog to Todo in Rafiki Nov 25, 2024
@sanducb sanducb moved this from Todo to In Progress in Rafiki Dec 3, 2024
@sanducb sanducb linked a pull request Dec 10, 2024 that will close this issue
6 tasks
@sanducb sanducb linked a pull request Dec 10, 2024 that will close this issue
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

2 participants