Skip to content

[TOOL-156] Unify Transform #306

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 42 commits into from
Nov 24, 2023
Merged

[TOOL-156] Unify Transform #306

merged 42 commits into from
Nov 24, 2023

Conversation

davidabram
Copy link
Member

@davidabram davidabram commented Nov 12, 2023 •
edited
Loading

closes #307

@davidabram davidabram marked this pull request as draft November 12, 2023 22:46
@davidabram davidabram changed the title Unify Transform [TOOL-156] Unify Transform Nov 13, 2023
davidabram and others added 23 commits November 13, 2023 13:56
@davidabram
added mr size
32470ac
@davidabram
hunks and codegen
323254e
@davidabram
getTransformDates
fdd9600
@Ante-Koceic
Using UTC values and added week
c0f08d2
@dejan-crocoder
Merge branch 'main' into feat/transform
185b5af
@dejan-crocoder
add getTimelineReviewDepth (incorrect implementation)
eae3a98
@dejan-crocoder
getTimelineApproved getTimelineReviewed
901af7d
@dejan-crocoder
getTimelineStartedCodingAt
7e1142e
@dejan-crocoder
add startedCodingAt, reviewDepth, approved and reviewed to run function
1547a93
@davidabram
very good code
bff9678
@davidabram
fix
f6e564f
@Ante-Koceic
Fixed tests
b68cbd6
@davidabram
Merge branch 'main' into feat/transform
197740c
@dejan-crocoder
add review time and review depth
a22ac85
@dejan-crocoder
add start pickup time
50552be
@dejan-crocoder
timeline reviewed and approved
2347788
@dejan-crocoder
add pickup and review times to date dereferentiation
78de8fd
@dejan-crocoder
specify columns and cleanup old stuff
8dba298
@dejan-crocoder
fix types and learn more why types arent attractive
aaa94c2
@Ante-Koceic
Added inserting data into database, lastUpdatedAt in map, missing ret…
1c49fde 
…urn values
@Ante-Koceic
Calculating time durations
fb8c797
@Ante-Koceic
Updated transform metrics data
91b282d
@davidabram
Merge branch 'main' into feat/transform
62caf13
@dejan-crocoder dejan-crocoder mentioned this pull request Nov 21, 2023
Closed
@davidabram davidabram marked this pull request as ready for review November 24, 2023 11:08
Ante-Koceic
Ante-Koceic approved these changes Nov 24, 2023
View reviewed changes
Copy link
Contributor

@Ante-Koceic Ante-Koceic left a comment

Choose a reason for hiding this comment

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

👍

@davidabram davidabram merged commit f387161 into main Nov 24, 2023
@davidabram davidabram deleted the feat/transform branch November 24, 2023 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ante-Koceic Ante-Koceic Ante-Koceic approved these changes

@ivke995 ivke995 ivke995 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[TOOL-156] Unify Transform
4 participants
@davidabram @Ante-Koceic @ivke995 @dejan-crocoder