Skip to content

Comments

Allow /conversations/<uuid> paths for legalhold tokens#2726

Merged
smatting merged 2 commits intodevelopfrom
allowlist-lhclients-followup
Sep 26, 2022
Merged

Allow /conversations/<uuid> paths for legalhold tokens#2726
smatting merged 2 commits intodevelopfrom
allowlist-lhclients-followup

Conversation

@smatting
Copy link
Contributor

@smatting smatting commented Sep 26, 2022

This PR:

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@smatting smatting temporarily deployed to cachix September 26, 2022 10:21 Inactive
@smatting smatting temporarily deployed to cachix September 26, 2022 10:21 Inactive
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Sep 26, 2022
@smatting smatting temporarily deployed to cachix September 26, 2022 10:24 Inactive
@smatting smatting temporarily deployed to cachix September 26, 2022 10:24 Inactive
@smatting smatting marked this pull request as ready for review September 26, 2022 10:24
@smatting smatting requested a review from pcapriotti September 26, 2022 10:25
Copy link
Contributor

@pcapriotti pcapriotti left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@smatting smatting merged commit 1414c07 into develop Sep 26, 2022
@smatting smatting deleted the allowlist-lhclients-followup branch September 26, 2022 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants