Skip to content

Consolidate DatabaseProxy abstraction layer #267

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

Closed
dmohns opened this issue Aug 28, 2024 · 2 comments · Fixed by #461
Closed

Consolidate DatabaseProxy abstraction layer #267

dmohns opened this issue Aug 28, 2024 · 2 comments · Fixed by #461
Assignees
Labels
Code base improvement Any improvements to the code base which makes it more maintainable or easier to use for users. enhancement New feature or request

Comments

@dmohns
Copy link
Member

dmohns commented Aug 28, 2024

Background:

In the multi-tenancy layout of the MPM there is some redundancy of User accounts and DatabaseProxy. User accounts "live" in the tenant databases. However, they need to get elevated and duplicated into DatabaseProxy. As a result the DatabaseProxy is essentially a UNION of all tenant user tables.

This is redundant and creates confusion.

Scope:

Please make User objects live at the MPM root. And use the DatabaseProxy work with the newly created User objects. Adapt services accordingly.

Definition of done:

  • The model DatabaseProxy and all related services and code is removed
  • The model User now lives at root level
@dmohns dmohns added enhancement New feature or request Code base improvement Any improvements to the code base which makes it more maintainable or easier to use for users. labels Aug 28, 2024
@beesaferoot beesaferoot self-assigned this Dec 15, 2024
@dmohns
Copy link
Member Author

dmohns commented Feb 20, 2025

Hey @beesaferoot

I think this was closed by mistake, right 😅 ? Can we re-open it?

@beesaferoot
Copy link
Contributor

Oh right, I changed the status to closed my project list. Sorry

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code base improvement Any improvements to the code base which makes it more maintainable or easier to use for users. enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants