Skip to content

Conversation

@mandres2015
Copy link
Collaborator

This pull request get the n (20 default) latest entries from an user

@mandres2015 mandres2015 force-pushed the TT-407-LIST-LATEST-V2-Time-Entries branch from f65527d to 56e4ced Compare November 22, 2021 20:26
Copy link
Collaborator

@josepato87 josepato87 left a comment

Choose a reason for hiding this comment

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

Left some comments

@mandres2015 mandres2015 force-pushed the TT-407-LIST-LATEST-V2-Time-Entries branch from 38162e7 to f323f37 Compare November 25, 2021 21:16
@mandres2015 mandres2015 force-pushed the TT-407-LIST-LATEST-V2-Time-Entries branch from f323f37 to 0c39250 Compare November 26, 2021 16:51
@mandres2015 mandres2015 force-pushed the TT-407-LIST-LATEST-V2-Time-Entries branch from 0c39250 to 826997b Compare November 26, 2021 16:53
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@scastillo-jp scastillo-jp self-requested a review November 26, 2021 19:55
@mandres2015 mandres2015 dismissed josepato87’s stale review November 26, 2021 19:55

I am discarding your review because the comments that you had left me have already been corrected and in the next ticket certain aspects of this ticket will be corrected

@mandres2015 mandres2015 merged commit f9e1403 into master Nov 26, 2021
@mandres2015 mandres2015 deleted the TT-407-LIST-LATEST-V2-Time-Entries branch November 26, 2021 19:55
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