[BUG] Fix configs to support multi-tenancy

Open
#346 0 comments 0 reactions 1 assignee View on GitHub

@roberlander2 is already working on this.

Since Jan 26, 2023.

  • #359 by @roberlander2 — closed without merging

Assessment

This issue has not been assessed yet.

Description

bug

Describe the bug
The current implementation of ApplicationConfig does not support multi-tenancy. Only one application config is used (based on environment variables), so any functions using data from an ApplicationConfig will not support multi-tenancy.

Expected behavior
Implement the configs collection solution from https://github.com/rokwire/building-block-template-go and add Type, AppID, and OrgID fields to support multi-tenancy. Also, a System flag should be added to configs to allow system-level users to control which configs may be modified by lower-level admins. SyncConfig and ManagedGroupConfig should both be stored in the configs collection.

Dominant language
Go
Stars
1
Forks
1
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from rokwire/groups-building-block

All issues in rokwire/groups-building-block

Similar issues

More Go issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.