ClockifyClient.FindAllTimeEntriesForProjectAsync "hydrated" parameter makes no sense

[topennanen]

As the method can only return unhydrated time entries (TimeEntryDtoImpl) , the parameter "hydrated" is useless.

Suggestion: Separate method for hydrated time entries, which uses appropriate method for the use time entry queries (ClockifyClient.FindAllHydratedTimeEntriesForUserAsync) and returns HydratedTimeEntryDtoImpl

[Morasiu]

I will check it with current API. Thanks for suggestion!

[Morasiu]

I see I already have FindAllHydratedTimeEntriesForUserAsync. Is it enough for your needs?

[topennanen]

Not really, no. The use case is finding all time entries for a task, regardless of the user, along with the names of the users who made those entries.

[Morasiu]

So a method like FindAllHydratedTimeEntriesForUserAsync, but FindAllHydratedTimeEntriesForTaskAsync?

[topennanen]

That, or FindAllHydratedTimeEntriesForProjectAsync for a wider usage in line with the current non-hydrated method.

[Morasiu]

Roger that, will do :)