Alya Abbott
Alya Abbott
@palashb01 The commits still don't look quite right:
@Joelute please address the failing tests, and post a comment when the PR is ready. Thanks!
I tried moving all the messages in a topic into an existing topic via the "Move messages" UI, and got the following error: ![Screen Shot 2022-10-25 at 10 05 53...
@laurynmm Is this PR ready for integration review from your perspective? I'll do another round of manual testing when that's the case. Thanks!
Looks good to me in manual testing! @timabbott this PR has been reviewed by @laurynmm and me -- please take a look.
Let's also discuss @laurynmm 's comment above live; I don't remember if there was some reason why it currently works the way it does.
Looks fine to me in English; not sure how to test anything else.
@amanagr Please add the "integration review" label whenever this PR is ready from your perspective. Thanks!
@timabbott Adding the integration review label because you've been reviewing it and your feedback above has been addressed.
No, "who has the authority to private message" is not a correct summary of "All direct message and group direct messages must include as sender or recipient at least one...