-
Notifications
You must be signed in to change notification settings - Fork 34
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
flag a challenge #1868
Comments
What is the process for removing this flag? Just thinking that if a user flags a challenge and then the issue is resolved. |
Oh, Also, I hope it will be clear that a challenge should only be flagged if the challenge is of very bad quality. I would hope that if a single user performs one task of a challenge and that one task had an error in it or incorrectly flagged an issue where it doesn't exist that they will not flag the challenge. How can they know that there is a real problem with a challenge unless they perform a number of tasks to verify that the issue exists across all (or most?) tasks in the challenge. I'm not suggesting that you require the user to do more than one task, but the text should make it clear that the challenge should not be flagged just because of one task that was bad. |
In the context of the description, I would not call it flag but report. In my view, a flag is only a visual indication. Here a report would really be created. |
@atiannicelli the issue being closed on the Github side would "unflag" the challenge. The outcome could be that the challenge is poorly enough designed that it should go away (be archived or even altogether deleted). Perhaps there's a GitHub label that we could use for that, which MapRoulette could pick up and trigger the needed operation. And you're correct, the interface needs to make it clear that this is a serious operation. I have thought about the term "Flag" vs "Report" and landed on "Flag" for now, but as @mcliquid also states, this may not convey that gravity. I figured "Report" sounded too..Harsh? But perhaps it's the better way to go. |
@mcliquid A deadline is nice in theory, but in practice it would depend on who can help adjudicate the issues that are reported. I'm thinking one thing we could do is to have a warning in the Challenge and associated Task screens that this challenge has been Flagged / Reported X days ago and to proceed with care. With a link to the Github issue perhaps for clarification. |
I added a template for the issue to be created to the report repo README |
A few thoughts on this as I gather feedback. First, I am leaning towards "Report" rather than "Flag". It conveys better that this is a serious action to take. I still don't quite like the implication that there is some entity / process behind this -- there isn't -- but I think "Report" is the better choice between the two options. Second, I don't favor the UX of an icon to represent both the action "Report a challenge" and the status "This challenge is reported". I think something like this would work, looking for opinions here (try and look beyond the bad graphics editing..): So in this graphic, small buttons would replace the "View Leaderboard" and "Get in Touch" text links, and another red-color button would open the reporting modal (as described above) Once a challenge is reported, the button would change to "Reported" and not be clickable. @ryanscherler may have thoughts on this? |
@mvexel I agree that less icons is better - labels are a good use here. I believe we already have labels in the system so this would be a good solution using existing UI elements. |
@ryanscherler thanks. Except these are meant as buttons. Maybe they do look more like labels, which is not what I intended. |
For the email we can use the following:
Then for the reporter
For the Challenge owner:
|
Resolved with #1933 |
I would like to introduce the capability for any logged-in user to flag a challenge. The flow would be as follows:
UI Flow
Conditions
Modal content
Button should only be clickable if text input is within character limits and the checkbox is selected.
The text was updated successfully, but these errors were encountered: