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

feature: temporalcloud_user_namespace_access #122

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Commits on Sep 24, 2024

  1. feature: temporalcloud_user_namespace_access

    This PR addresses #119, #116, and #115 by decoupling the definition of user accesses from the user itself, via a new resource: `temporalcloud_user_namespace_access`. This resource is intended to provide a many-to-many mapping between namespaces and users. Under the hood, this resource is manipulating a single User object via the API (as the underlying data model stashes all namespaces accesses on the user object), while also preserving the invariant that adding or removing a single user from a single namespace won't obliterate the list of permissions that a user has.
    
    I do intend to write some more tests but I wanted to get this out quickly for review for some fast feedback before I write a bunch of tests that exercise things that might change in review.
    swgillespie committed Sep 24, 2024
    Configuration menu
    Copy the full SHA
    678da87 View commit details
    Browse the repository at this point in the history