Skip to content

Conversation

CuddlyBunion341
Copy link
Contributor

@CuddlyBunion341 CuddlyBunion341 self-assigned this Aug 29, 2025
@CuddlyBunion341 CuddlyBunion341 changed the title Optional session comments Optional timer session comments Aug 29, 2025
Copy link
Member

@coorasse coorasse left a comment

Choose a reason for hiding this comment

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

Yes, please. @schmijos are you ok with it?

Copy link
Member

@schmijos schmijos left a comment

Choose a reason for hiding this comment

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

I don't think this is a good idea. @samuelsteiner will come and ask you what you did when invoicing your hours.

Copy link
Member

@rnestler rnestler left a comment

Choose a reason for hiding this comment

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

It's only annoying and without much value for some tickets like company meeting, where I usually just type "attend".

For the others I think comments are valuable.

We could add logic to have default comments for some special tickets like the company meeting?

@schmijos
Copy link
Member

"attend" is way more information than "" already.

@CuddlyBunion341
Copy link
Contributor Author

What happens if you book time on multiple tickets at once?
What if the defaults conflict?

So basically set default on ticket.
If creating a new session with the ticket, autofill the placeholder?

@schmijos
Copy link
Member

We've been thinking about personal preferences anyways since a long time. But there might be an easier solution.

You could simply assume the last timer session entry comment for the current ticket if it's the same. So assuming Alessandro works on one ticket X, then every time he's working on this ticket, the comment would be prefilled with what he entered last time.

The condition I would have to this feature is that on focus the whole default text is marked so that I can easily enter my own.

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

Successfully merging this pull request may close these issues.

4 participants