addons icon indicating copy to clipboard operation
addons copied to clipboard

Convert ReportAbuseButton to setUIState

Open kumar303 opened this issue 6 years ago • 5 comments

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 with setUIState.

In https://github.com/mozilla/addons/issues/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

kumar303 avatar Sep 24 '18 14:09 kumar303

Some complimentary clean-up will happen in https://github.com/mozilla/addons/issues/12655

kumar303 avatar Nov 12 '18 23:11 kumar303

hello @kumar303 , I'd like to take this up. Is this issue still available?

grlwholifts avatar Aug 21 '19 17:08 grlwholifts

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 contrib: welcome patches as those are mostly ready to go.

kumar303 avatar Aug 28 '19 17:08 kumar303

Okay @kumar303! Will do that 🙇

grlwholifts avatar Aug 29 '19 04:08 grlwholifts

Old Jira Ticket: https://mozilla-hub.atlassian.net/browse/ADDFRNT-154

KevinMind avatar May 03 '24 18:05 KevinMind