Skip to content
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

Challenges are often kept as active despite having no tasks #1092

Open
matkoniecz opened this issue Dec 8, 2023 · 6 comments
Open

Challenges are often kept as active despite having no tasks #1092

matkoniecz opened this issue Dec 8, 2023 · 6 comments
Assignees

Comments

@matkoniecz
Copy link

see for example https://maproulette.org/browse/challenges/40603

https://maproulette.org/challenge/41154

https://maproulette.org/challenge/40667

@KasperFranz
Copy link

@matkoniecz
Copy link
Author

may be caused by changing task state via API, not user interface?

Maybe I trigger it when last task is set to "disabled" not "fixed"?

@matkoniecz
Copy link
Author

@ljdelight
Copy link
Contributor

It does look like this is caused by tasks that are Disabled, and that prevents the Challenge from being completed. For challenge 40109: I don't see anything in the UI that indicates disabled tasks; the GET challenge API call response shows a completionPercentage of 100 and yet the challenge remains active; then the GET challenge data shows 12 tasks, 9 fixed, 3 disabled, 11 deleted.

I bet there's some task completion logic that needs to be updated to account for disabled tasks, or perhaps it's intentional and should be more obvious in the UI.

@matkoniecz
Copy link
Author

Disabled tasks definitely should not keep a challenge open. If they do, what is the point of having them?

(in fact - what is the point of having both disabled and deleted?)

@mvexel
Copy link
Member

mvexel commented Aug 20, 2024

disabled may make sense to a challenge author, to have tasks that they temporarily want to have not included but for whatever reason they do not want to delete, because they may want to re-enable them later.

From an end user perspective this status really does not have any meaning.

So what we'll do is

  • On the front end, do not count the disabled tasks in the totals. So in the above example, the total task count that the user sees would be 9 and not 12 (separate ticket to be created for that)
  • On the backend, once a challenge has only deleted or disabled tasks left, we mark it as completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants