Skip to content

Comments

Allow deletion of MLS team conversations#2733

Merged
smatting merged 3 commits intodevelopfrom
FS-1002-delete-mls-team-conv
Sep 28, 2022
Merged

Allow deletion of MLS team conversations#2733
smatting merged 3 commits intodevelopfrom
FS-1002-delete-mls-team-conv

Conversation

@smatting
Copy link
Contributor

@smatting smatting commented Sep 28, 2022 •
edited
Loading

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 28, 2022 10:51 Inactive
@smatting smatting temporarily deployed to cachix September 28, 2022 10:51 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 28, 2022
@smatting smatting temporarily deployed to cachix September 28, 2022 12:29 Inactive
@smatting smatting temporarily deployed to cachix September 28, 2022 12:29 Inactive
@smatting smatting changed the title Fix bug: Cannot delete MLS conv Allow deletion of MLS team conversations Sep 28, 2022
@smatting smatting marked this pull request as ready for review September 28, 2022 12:31
@smatting smatting temporarily deployed to cachix September 28, 2022 12:31 Inactive
@smatting smatting temporarily deployed to cachix September 28, 2022 12:31 Inactive
@smatting smatting merged commit 7cce247 into develop Sep 28, 2022
@smatting smatting deleted the FS-1002-delete-mls-team-conv branch September 28, 2022 12:46
@zebot zebot mentioned this pull request Oct 4, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stefanwire stefanwire stefanwire approved these changes

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@smatting @stefanwire @zebot