This repository was archived by the owner on Feb 8, 2024. It is now read-only.
[v11] [Connect] Add Document Access Requests (#1203)#1252
Merged
avatus merged 2 commits intoteleport-v11from Oct 12, 2022
Merged
[v11] [Connect] Add Document Access Requests (#1203)#1252avatus merged 2 commits intoteleport-v11from
avatus merged 2 commits intoteleport-v11from
Conversation
* init webapps.e ref * update web.e * added imported components * udpated to correct imported types * webapps.e update * readme update and lint fix * removed extra console logs * sub head fix for webe * reattached webape head * fixed tests * added access request change confirm dialog * null check * webappe * count to accessRequestSerivce, import/export names, small fixes * fixed imports and createrequest params * updated imports from removed containers in teleterm-e * optional params for reuse outside of ClusterLogout * webapps.e ref update * update webapps.e ref * webapps.e ref update * webapps.e ref update and navigation item update * webapps.e update * align sync button center to match access request size * added accessRequestsSerice tests * updating webapps.e * created accessRequests object on workspace type and cleanup * added requestId to documentaccessrequest type and webapps.e update * protobuf updates changing created/expires from string to timestamppb * webappse update and review feedback * updated protos with comments * changing delete/review access request responses to void * updated getaccessrequests to not include ID * protobuf updates * webappse update * update to resource tables styles * update to standard access_request_id field in RPCs * webappse upddate * updated tests and added types * updated webappse * clusterService method for getrequestableroles, retryWithRelogin for fetch, webappse update * webappse update * story fixes * added kubes support to tsh * proto files * update webappse * Provide `WorkspaceContext` for documents in each workspace * updating webappse * webappse update * accessrequestservice fix for resoruces * updated webapps.e after master merge * removed extra data from persisted state for assumed access requests * update webappse * protobuf files * lint fixes * adding getWorkspaces stub to fix tests * webappse update * webappse snapshots update * protobuf updates Co-authored-by: Grzegorz Zdunek <grzegorz.zdunek@goteleport.com>
gzdunek
approved these changes
Oct 12, 2022
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
backport #1203
Will update e ref once https://github.com/gravitational/webapps.e/pull/407 gets in