search

Leantime / leantime

Leantime is a goals focused project management system for non-project managers. Building with ADHD, Autism, and dyslexia in mind.
https://leantime.io
GNU Affero General Public License v3.0
4.43k stars 580 forks source link

[BUG] Adding time in the timesheet deletes existing rows #2597

Open lilosti opened 1 month ago

lilosti commented 1 month ago

If you have problems with the installation please use our community forum

What set up are you using

Describe the bug A clear and concise description of what the bug is.

To Reproduce Steps to reproduce the behavior:

  1. Go to timesheet weekview
  2. Add a row with registrations for 4 days in a week (1 hour each day) for projekt X Todo Y
  3. Add a new registration to projekt X row Y for 1 hour
  4. Press save
  5. See that the first registration you made is overwritten => you just lost the existing registrations

Expected behavior 2 choices:

  1. you cannot choose a todo there is already a registration on in weekview
  2. if you choose a todo that aldready has a registration the registration should be added. deleting registrations is never correct behavior.

Leantime Version Which Leantime version are you using new

Server Apache, nginx, IIS etc

PHP / MySQL Version Please list the PHP and Mysql version you are using

Additional context Add any other context about the problem here.

marcelfolaron commented 1 month ago

Thanks for the submission. The value gets updated as you have called out. So it's less of an overwrite as it is an update of the existing number of hours to the new value.

Now I agree that there is a UX issue here in that we should now allow creating 2 rows of the same ticket in one week. I think easiest solution here would be to remove the todo from the selection for that week. Would that solve the issue?