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

[Bug]: version history in reviewer's pages don't have "forwarded to legal" #15327

Open
1 task done
ioanarusiczki opened this issue Feb 3, 2025 · 3 comments
Open
1 task done
Assignees

Comments

@ioanarusiczki
Copy link

ioanarusiczki commented Feb 3, 2025

What happened?

After #15306 I've been looking around on Forward to Legal feature and noticed some changes in rev tools version history

STR:

  1. A version is reported for abuse report and flagged for HR in the Manual Review Queue
  2. Version is forwarded to Cinder's Legal Queue
  3. Decision is to disable content with Illegal policy

What did you expect to happen?

On dev version's history does not have "Forwarded to Legal" https://reviewers.addons-dev.allizom.org/en-US/reviewers/review-listed/635774
Atm stage has this https://reviewers.addons.allizom.org/en-US/reviewers/review-listed/2245117

also on dev there's "force disabled" presentin version's history, unlike -stage where this is only available in " Add-on important changes history " (which I find ok)

dev

Image

stage

Image

Is there an existing issue for this?

  • I have searched the existing issues

┆Issue is synchronized with this Jira Task

@ioanarusiczki
Copy link
Author

Note: checking admin activity log on dev - no problem here, I see the action "forwarded for legal review"

@diox
Copy link
Member

diox commented Feb 4, 2025

Need to investigate, if we are storing the activity log maybe we are no longer associating it with the version? That would be bad.

@diox
Copy link
Member

diox commented Feb 4, 2025

Confirming: while we do reword an ActivityLog, it's not tied to a specific version - the entire add-on is forwarded to legal - so we don't see it in the version history...

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

No branches or pull requests

3 participants