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

Remove SiteUsers concept #4339

Closed
6 tasks done
drewbo opened this issue Dec 19, 2023 · 0 comments · Fixed by #4683
Closed
6 tasks done

Remove SiteUsers concept #4339

drewbo opened this issue Dec 19, 2023 · 0 comments · Fixed by #4683
Assignees

Comments

@drewbo
Copy link
Contributor

drewbo commented Dec 19, 2023

Prior to the introduction of organizations, users were given access to a site via the SiteUser model. We need to remove this concept (all models, tables, endpoints, and related code) from the database, API, and application

Acceptance Criteria

  • SiteUser model, Site Users column is removed in production database via migration
  • Authorizer and model scope code for builds (.forSiteUser) and for site (.forUser) are updated to only reference organization membership as the site access condition
  • SiteUser audit event is removed
  • SiteUser Redux actions, reducers, and associated front end code is removed
  • Rewrite loadBuildUserAccessToken to check organization users rather than site.Users for a valid access token
  • Remove or rewrite all associated tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant