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

fix bug that user in url was not selected #7722

Merged
merged 3 commits into from
Mar 27, 2024
Merged

Conversation

dieknolle3333
Copy link
Contributor

@dieknolle3333 dieknolle3333 commented Mar 26, 2024

Bug: Clicked user in overview was not preselected in detail view. The reason seems to be that the data is fetched while the state is not yet updated.

Steps to test:

  • Go to time tracking overview and click details for a specific user
  • make sure this user is selected in the detail view, in the dropdown

Issues:


(Please delete unneeded items, merge only when none are left open)

  • Updated changelog
  • Removed dev-only changes like prints and application.conf edits

Copy link
Member

@philippotto philippotto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me!

@philippotto philippotto enabled auto-merge (squash) March 27, 2024 09:33
@philippotto philippotto merged commit 6836f46 into master Mar 27, 2024
2 checks passed
@philippotto philippotto deleted the fix-time-tracking-user branch March 27, 2024 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants