Add clean-issue-filters feature

[notlmn]

Closes #2016.

Test

• With no projects or milestones: https://github.com/ramlmn/serv/pulls.
• With repo level projects and no milestones: https://github.com/notlmn-dump/test/pulls.
• With milestone and no projects: https://github.com/notlmn-dump/now-express-test/pulls.
• With org level projects and not milestones: https://github.com/notlmn-dump/opev-gen-googio/pulls (needs org access for oAuth token).

[notlmn]

Also thought of adding support for repo deployment status, but that requires another API request (one to get latest commit, and one to get its deployment status). And targeting status dropdown is a bit of problem too.

[fregante]

Can you add the feature to the readme and add yourself to CODEOWNERS?

1. Zoom at 200% (or have a retina display)
2. Reuse this format and its arrows (copy-paste them): https://user-images.githubusercontent.com/1402241/55069759-bceaf080-50bf-11e9-84d0-7707de2eb9e9.png

Also it would be good to cache the result of this, per-repo, especially with #2109 coming :)

If not impossible, it would be nice to also drop the Checks selector. Does the API include that?

[notlmn]

If not impossible, it would be nice to also drop the Checks selector. Does the API include that?

Took me a while to debug, but that's part of filter-pr-by-build-status. The dropdown doesn't exist until that features adds it, maybe it's better to handle this in that feature.

---

Also it would be good to cache the result of this, per-repo, especially with #2109 coming :)

Let's deal with that in a separate PR. Or will have to wait until #2109 is merged.

[fregante]

The dropdown doesn't exist until that features adds it, maybe it's better to handle this in that feature.

D’oh, I forgot about that

[fregante]

Apparently even if I have a personal project, public, with an issue from the current repo inside, the dropdown is still empty for me.

I dropped this

[FloEdelmann]

Thanks @notlmn! 😊

[fregante]

It does look much better now!

[notlmn]

If not impossible, it would be nice to also drop the Checks selector. Does the API include that?

Took me a while to debug, but that's part of filter-pr-by-build-status. The dropdown doesn't exist until that features adds it, maybe it's better to handle this in that feature.

Opened #2127 to track this.

[yakov116]

@notlmn nothing major but

❌ clean-issue-filters
content.js:2111 TypeError: Cannot read property 'parentElement' of null
    at clean_issue_filters_init (content.js:6423)
    at async features_run (content.js:2106)

When a repo has not Issues or Pull requests