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

[WPB-9685] don't react with "legalhold already disabled" when on pending state #4104

Merged
merged 4 commits into from
Jun 24, 2024

Conversation

MangoIV
Copy link
Contributor

@MangoIV MangoIV commented Jun 24, 2024

https://wearezeta.atlassian.net/browse/WPB-9685

Checklist

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

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jun 24, 2024
@MangoIV MangoIV force-pushed the mangoiv/legalhold-request-is-not-deleted branch from 3cc4ed5 to a721a33 Compare June 24, 2024 10:05
@MangoIV MangoIV changed the title [chore] add test that confirms behaviour of LH from the issue [WPB-9685] don't react with "legalhold already disabled" when on pending state Jun 24, 2024
@MangoIV MangoIV marked this pull request as ready for review June 24, 2024 11:28
@MangoIV MangoIV requested a review from fisx June 24, 2024 11:28
changelog.d/3-bug-fixes/WPB-9685 Outdated Show resolved Hide resolved
integration/test/Test/LegalHold.hs Outdated Show resolved Hide resolved
Co-authored-by: Akshay Mankar <akshay@wire.com>
@MangoIV MangoIV merged commit 87b8f96 into develop Jun 24, 2024
10 checks passed
@MangoIV MangoIV deleted the mangoiv/legalhold-request-is-not-deleted branch June 24, 2024 13:20
@echoes-hq echoes-hq bot added the echoes: unplanned Any work item that isn’t part of the product or technical roadmap. label Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
echoes: unplanned Any work item that isn’t part of the product or technical roadmap. 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.

4 participants