maproulette / maproulette/maproulette-frontend
tasks modified by challenge owner are initially listed in history tab as modified by last active mapper
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 167
- Forks
- 39
- Avg merge
- 16h 34m
- Merged PRs (30d)
- 11
Description
Describe the bug
tasks modified by challenge owner are initially listed in history tab as modified by last active mapper
looks to be UI bug as reload removes info who modified them (what may be also a bug)
Expected behavior
Person who manually edited task state is listed in history view as one who modified it
To Reproduce
- Go to task set as false positive in inspect task mode
- Click modify task button
- Change status to disabled
- Click save
- look at history tab
Screenshots
one after stage 5 (I am not mcliquid and I edited it to disable it)
one after reloading page
Desktop (please complete the following information):
- OS: Lubuntu 20.04
- Browser Firefox
- Version 120.0.1
Additional context
I guess if many people can have admin rights over challenge then having no log who edited it would be more annoying
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Reproduce this in inspect task mode by modifying a task from false positive to disabled, then check the history tab before and after reloading. Trace the UI flow behind the modify task action and history view to find where the editor identity is displayed or lost. Done means the person who manually changed the task state is shown both immediately and after reload.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100