Add disclaimer on the skill filter in the issue finder

[dhruvkb]

Description

Not all issues have skills marked on them, especially if they are simple issues that do not require proficiency in any specific framework or language. Those issues are gone when filtering by skill in the Issue Finder. This behaviour should be explained to the user on the page.

Reproduction

1. Open the Issue Finder.
2. See all issues.
3. Filter by a specific skill.
4. See a much much smaller subset of issues, and more likely the harder ones.
5. That's not good.

Expectation

This anomalous behaviour should be well documented.

Alternatives

Alternatively, we can collectively decide that every issue should have a skill, regardless of the level of proficiency of said skill required to resolve the issue (might tie into creativecommons/ccos-scripts#86).

Resolution

• I would be interested in resolving this bug.

[CodeSarthak]

Hey @dhruvkb ,

I think switching to a system where we have to assign a skill to each issue might be a good choice in the long run but it will take time to switch over and it would have to be very well documented and prominent in the docs.

Therefore, I believe that a short term fix might be to simply document this behavior. Then, we can open up a new issue and work towards a more permanent/long-term fix.

How does that sound?

P.S. : This is my first issue here 😁

[dhruvkb]

Yes, we should document this for now.

[CodeSarthak]

Great, should I go ahead and do that?

[c99SRS]

Hi @dhruvkb, can I work on this?

[dhruvkb]

@c99SRS in #527, @CodeSarthak has already made a PR to fix this, but I've added some pointers for improvement to that. Could you open a PR with only the suggested changes (so that I can merge both independently)?

[c99SRS]

HI @dhruvkb , Could you please tell any specific UI for this or is there any particular section we have to make the change?
But I have made changes to this. Attached the screenshot also. Please let me know if it looks good.

[dhruvkb]

^ Can you put the disclaimer here instead of the header region?

[c99SRS]

@dhruvkb

^ Can you put the disclaimer here instead of the header region?

Don't you think it would be so much text on that particular section? I made changes also. Take a look and provide some feedback. If it looks odd, then I can suggest that text we can add in the header part 2-3 lines below from the current header part and we will mention that "Disclaimer" followed by those texts. Let me know your view on this?

[dhruvkb]

How about we add an asterisk * to the Skill set header and put that text in small font at the bottom of the page?

[c99SRS]

How about we add an asterisk * to the Skill set header and put that text in small font at the bottom of the page?

Yes. we can try out that one.

[c99SRS]

HI @dhruvkb , please take a look, made some changes. Attached the screenshot.

[c99SRS]

Hey @dhruvkb , I am raising one PR, if you are ok with this.

[c99SRS]

Hi @dhruvkb , could you please merge my PR?

[dhruvkb]

@c99SRS I'll review and merge it soon. You can continue to work on other issues in the meantime!