Skip to content
This repository has been archived by the owner on May 13, 2024. It is now read-only.

Bugs in new timed release #799

Closed
c0rydoras opened this issue Feb 9, 2023 · 10 comments
Closed

Bugs in new timed release #799

c0rydoras opened this issue Feb 9, 2023 · 10 comments
Labels
Milestone

Comments

@c0rydoras
Copy link
Contributor

Timesheet

  • This got fixed while i was testing (see fix(tracking): allow null values on remaining effort for reports timed-backend#928)
    • Creating a new report does not work, the problem seems to be, that remaining-effort is null
      • {"errors":[{"detail":"This field may not be null.","status":"400","source":{"pointer":"/data/attributes/remaining-effort"},"code":"null"}]}
    • The same applies to changing a reports customer/project/task (i tested this on reports created using the generate timesheet button)
  • Rescheduling
    • When rescheduling a day with verified events, it works, but you get this error message: image
      • this also applies to the current timed.
      • the diagram (weekly-overview) also does not update, even though the timesheet was rescheduled, after a refresh/switching to another day/pressing the save button again the diagram updates

Absences

  • After adding one or multiple absences they are not displayed until a refresh/switching day
  • After removing an absence, when trying to create a new absence (on the same day), the current day is selected and disabled, and therefore cannot be deselected.

Analysis

  • When sorting the Total time changes to 0 and then back. It does always stay the same since its the total.

Activities

  • The record-button has no icon
  • In the "Activities contain unknown tasks" modal the "thats fine" button is missing the margin right
    • test: image
    • normal: image
@c0rydoras c0rydoras added the bug label Feb 9, 2023
@derrabauke
Copy link
Contributor

Thanks for the feedback! 🙏

Activities

  • Is this the record button you are refer to? The button seems to be visible on my side. Maybe try clearing the cache once.
    image

@c0rydoras
Copy link
Contributor Author

Sadly that did not resolve the issue. It still remains without an icon for me.

@c0rydoras
Copy link
Contributor Author

In this case the icon is not just missing for me but also for other people.

@trowik
Copy link
Member

trowik commented Feb 10, 2023

similar issue with the record button:

image

@derrabauke
Copy link
Contributor

@trowik @c0rydoras Which browser do you use?

Can't reproduce on Chromium (Version 109.0.5414.119) nor FF (109.0.1). 🤷‍♂️

@trowik
Copy link
Member

trowik commented Feb 10, 2023

@derrabauke using Firefox 109.0.1 as well...
I'll check the behaviour with other browsers later

@c0rydoras
Copy link
Contributor Author

@derrabauke
I tested it on Chrome (Version 109.0.5414.119 (Official Build) (64-bit))
and Firefox (Version 109.0.1)

@derrabauke derrabauke added this to the v3 release milestone Feb 14, 2023
@derrabauke
Copy link
Contributor

Timesheet

  • Rescheduling

    • When rescheduling a day with verified events, it works, but you get this error message: image

      • this also applies to the current timed.
      • the diagram (weekly-overview) also does not update, even though the timesheet was rescheduled, after a refresh/switching to another day/pressing the save button again the diagram updates

Splitting it up for better issue tracking

@derrabauke
Copy link
Contributor

Absences

  • After adding one or multiple absences they are not displayed until a refresh/switching day
  • After removing an absence, when trying to create a new absence (on the same day), the current day is selected and disabled, and therefore cannot be deselected.

Splitting it up for better issue tracking

@derrabauke
Copy link
Contributor

Everything got resolved. Please check next deployment (tomorrow) for regressions. ;)

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

No branches or pull requests

3 participants