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

Workspace trust changes #119017

Merged
merged 6 commits into from
Mar 16, 2021
Merged

Workspace trust changes #119017

merged 6 commits into from
Mar 16, 2021

Conversation

lszomoru
Copy link
Member

  • Use separate promises for soft/modal notifications
  • Add the capability to customise workspace trust dialog buttons

@lszomoru lszomoru added the workspace-trust Trusted workspaces label Mar 15, 2021
@lszomoru lszomoru added this to the March 2021 milestone Mar 15, 2021
@lszomoru lszomoru requested a review from sbatten March 15, 2021 20:05
@lszomoru lszomoru self-assigned this Mar 15, 2021
Copy link
Member

@sbatten sbatten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see comments. minor changes, but primarily approved.

@lszomoru lszomoru enabled auto-merge (squash) March 16, 2021 10:09
@lszomoru lszomoru merged commit 149a8b7 into main Mar 16, 2021
@lszomoru lszomoru deleted the lszomoru/workspace-trust branch March 16, 2021 10:18
lramos15 pushed a commit that referenced this pull request Mar 16, 2021
* Add dialog button customisation and reject promise if cancelled
* Use different promises to modal/soft requests
@github-actions github-actions bot locked and limited conversation to collaborators Apr 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
workspace-trust Trusted workspaces
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants