-
Notifications
You must be signed in to change notification settings - Fork 41
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
Convert ReportAbuseButton to setUIState #2060
Comments
Some complimentary clean-up will happen in #12655 |
hello @kumar303 , I'd like to take this up. Is this issue still available? |
Hi @ruchikabgosain, thanks for your interest. This isn't a good issue to work on because abuse reporting might move away from https://addons.mozilla.org/ and into Firefox itself (see https://bugzilla.mozilla.org/show_bug.cgi?id=1540175). However, we aren't sure yet. I suggest looking at |
Okay @kumar303! Will do that 🙇 |
Old Jira Ticket: https://mozilla-hub.atlassian.net/browse/ADDFRNT-154 |
For QA: ensure the Report abuse button on the add-on detail page still works without regressions.
The
ReportAbuseButton
relies on custom actions to set internal state. These actions should be removed and replaced withsetUIState
.In #2059, we will potentially have two Report abuse buttons on the same page. To support that we would need to enhance the current actions/reducers or just switch to
setUIState()
and set the component ID appropriately. Let's do the latter.┆Issue is synchronized with this Jira Task
The text was updated successfully, but these errors were encountered: