Added the ability to disable group similar time entries#1054
Added the ability to disable group similar time entries#1054utlark wants to merge 2 commits intosolidtime-io:mainfrom
Conversation
|
thanks for the contribution. a few notes:
|
|
i just realized the TimeEntryMassActionRow is used in the Reporting as well. We can just add a "settings" slot to the component and only pass it in Time.vue. The useStorage can stay in the main application then and can be passed down like the PR already does. |
|
Alright, I’ll try to move the setting closer to the Time view. I’ll also adjust how storage is handled (key, naming, extracting it into a separate file) and replace the ternary with a more readable early return. Regarding the tests — without Observations by case:
I agree that waiting by timeout is more of a workaround, but I was not able to find a stable alternative based on waiting for specific states. I’m not very familiar with writing Playwright tests, so I might be missing a better approach here. |
What does this PR do?
Adds the ability to disable grouping similar time entries. #445
Checklist (DO NOT REMOVE)