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

[WIP] [DO NOT MERGE] Unblock legalhold #824

Closed
wants to merge 5 commits into from

Conversation

fisx
Copy link
Contributor

@fisx fisx commented Aug 8, 2019

Revert "Block legalhold on stern / backoffice. (#823)"

This reverts commit 88cfe14 and
unblocks the legalhold feature.

Copy link
Contributor Author

@fisx fisx left a comment

Choose a reason for hiding this comment

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

we can merge this to develop iff we are ready to enable legalhold in production.

(hm, i wanted to request changes here to block the PR, but appearently i can only do that if i'm not the author. what the?)

@fisx fisx changed the title Unblock legalhold [WIP] Unblock legalhold Aug 8, 2019
@fisx fisx changed the title [WIP] Unblock legalhold [DO NOT MERGE] Unblock legalhold Aug 9, 2019
@fisx fisx force-pushed the fisx/unblock-legalhold-feature branch from a8bc8e3 to 8d7baaa Compare August 9, 2019 13:07
fisx added 5 commits September 5, 2019 09:33
This helps customer support to understand errors they run into without
consulting the server logs.  Stern is an internal tool, so it's ok to
be a little more generous with information on internal errors.
This may have prevented the bug fixed in
#844.
This reverts commit 88cfe14 and
unblocks the legalhold feature.
@fisx fisx force-pushed the fisx/unblock-legalhold-feature branch from 8d7baaa to c6c82e0 Compare September 5, 2019 08:01
@fisx fisx changed the title [DO NOT MERGE] Unblock legalhold [WIP] [DO NOT MERGE] Unblock legalhold Sep 5, 2019
@fisx fisx self-assigned this Sep 11, 2019
@fisx
Copy link
Contributor Author

fisx commented Sep 11, 2019

Included in #845 via 13bcf62

@fisx fisx closed this Sep 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant