Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@apoorv1316
Copy link
Collaborator

Closes #1746

@apoorv1316 apoorv1316 requested a review from nisusam April 3, 2024 07:38

joins_clause, group_field = case group_by
when :client
[[:project, :client], "clients.id"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Fix this

@apoorv1316 apoorv1316 force-pushed the time-enntry-group-duration branch from 01b3ed6 to 1e05939 Compare April 3, 2024 08:27
@apoorv1316 apoorv1316 force-pushed the time-enntry-group-duration branch from 0deb84c to 27fb639 Compare April 9, 2024 04:12
@apoorv1316 apoorv1316 requested a review from nisusam April 9, 2024 04:27
id_field = GROUP_BY_INPUT_TO_GROUP_ID_FIELD[group_by.to_s]

grouped_data = es_response.group_by { |entry| entry[label_field] }.map do |label, entries|
id = entries.first[id_field]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Fix indentation

@apoorv1316 apoorv1316 merged commit 42de04f into develop Apr 9, 2024
@apoorv1316 apoorv1316 deleted the time-enntry-group-duration branch April 9, 2024 06:59
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.

Time entry total should remain consistent throughout on reports page

3 participants