-
Notifications
You must be signed in to change notification settings - Fork 529
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
add logging for review replies + test #20689
Conversation
With dev replies, there is no log for deleting them. If a dev deletes a reply and then re-replies to that same comment, it's considered an edit, is this normal behaviour? |
It's a little unintuitive I agree, but the reasoning is that we a) want to keep deleted replies in general, but b) want at most 1 reply per rating, so a reply would just edit (and undelete) the instance. |
btw, you've got a conflict now because mat just landed a patch that also added an activity action - he's taken id=142 |
Bumps [protobuf](https://github.com/protocolbuffers/protobuf) from 4.22.3 to 4.23.0. - [Release notes](https://github.com/protocolbuffers/protobuf/releases) - [Changelog](https://github.com/protocolbuffers/protobuf/blob/main/generate_changelog.py) - [Commits](protocolbuffers/protobuf@v4.22.3...v4.23.0) --- updated-dependencies: - dependency-name: protobuf dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Quế Tùng <best.cloney.1301@gmail.com> Co-authored-by: Cát Thành <catthanh192@gmail.com>
Co-authored-by: Quế Tùng <best.cloney.1301@gmail.com>
Co-authored-by: Selim Şumlu <selim@sum.lu> Co-authored-by: Grk <gu0townkg@relay.firefox.com>
Co-authored-by: Balázs Meskó <meskobalazs@mailbox.org>
Co-authored-by: Théo Chevalier <theochevalier@pm.me>
Co-authored-by: h8211147g <h8211147g@outlook.com>
Co-authored-by: Michele Rodaro <michele.rodaro@mozillaitalia.org>
Co-authored-by: Michele Rodaro <michele.rodaro@mozillaitalia.org>
Co-authored-by: Francesco Lodolo [:flod] <flod+pontoon@mozilla.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ozilla#20688) * Record why a version was flagged for human review when that happens
Bumps [terser](https://github.com/terser/terser) from 5.17.1 to 5.17.3. - [Changelog](https://github.com/terser/terser/blob/master/CHANGELOG.md) - [Commits](terser/terser@v5.17.1...v5.17.3) --- updated-dependencies: - dependency-name: terser dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Lan Glad <upwinxp@gmail.com>
there's still a conflict |
Fixes mozilla/addons#9039
Remove the blocking of ADD_RATING logging for review replies. Update the associated test name & assertion.