Report infringing content within app

[miikkatu]

This is a suggested implementation for #909.

This is work-in-progress, as there is a CORS issue when posting the report, as discussed with @maximest-pierre.

[lbry-bot]

	Warnings
⚠️	📄 Changelog Entry Missing - Please add a changelog entry for your changes.

Generated by 🚫 dangerJS

[neb-b]

This is looking good, a couple comments.

Most of this should live in redux. We shouldn't be making api calls in in the component to keep things simple. You should also avoid mixing props and internal state like you are doing in this line:
identifier: this.props.params.uri !== undefined ? this.props.params.uri : '',

I propose creating a report reducer. Which will handle fetch errors, and loading while the api call is being made. The form values don't need to be stored in it, so Formik would be a good solution for this, similar to how we do it with the shapeshift component.

[miikkatu]

identifier: this.props.params.uri !== undefined ? this.props.params.uri : '',

I think in this case setting the state from props is ok, because it's like seed data. The form is generic so that the identifier can be uri but also something else as the UI info text says (Name, claim ID, or another way for us to identify the infringing content.).

This reporting function should eventually be generic, so that it could also be opened from sidebar. And when it's opened from there, the uri param will not be in the props, and the form will set the field as empty.

[miikkatu]

I propose creating a report reducer. Which will handle fetch errors, and loading while the api call is being made. The form values don't need to be stored in it, so Formik would be a good solution for this, similar to how we do it with the shapeshift component.

Looks like this would require more time than I currently have, unfortunately. Hopefully someone else can continue from here.

[tzarebczan]

@miikkatu thanks for your work so far, we've sent you a tip!

@btzr-io do you think this is something you could pick up? It's not urgent, but let me know if you are interested.

[alyssaoc]

closing, looking at a new direction/solution

[neb-b]

@miikkatu We may come back to this. I will re-open if we decide to go with this solution.