Skip to content
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.

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

feat: Add transformToV1Response & transformToV2Response for PlatformProject #2942

Merged
merged 1 commit into from
May 3, 2024

Conversation

SahilK-027
Copy link
Contributor

Description

As per the discussion in thread: https://fylein.slack.com/archives/C06M78V7JKD/p1713850203385719

Optional fields from ProjectV2

export interface ProjectV2 {
-  ap1_email?: string;
-  ap1_full_name?: string;
-  ap2_email?: string;
-  ap2_full_name?: string;
-  project_approver1_id?: string;
-  project_approver2_id?: string;
}
+ Created PlatformProject interface
+ Add transformToV1Response method
+ Add transformToV2Response method

Clickup

Please add link here

@SahilK-027 SahilK-027 self-assigned this May 3, 2024
@SahilK-027 SahilK-027 requested a review from Chethan-Fyle as a code owner May 3, 2024 06:52
Copy link

ellipsis-dev bot commented May 3, 2024

Your free trial has expired. To continue using Ellipsis, sign up at https://app.ellipsis.dev for $20/seat/month. If you have any questions, reach us at [email protected]

@github-actions github-actions bot added the size/M Medium PR label May 3, 2024
Copy link

github-actions bot commented May 3, 2024

Unit Test Coverage % values
Statements 95.88% ( 18567 / 19363 )
Branches 91.11% ( 9822 / 10780 )
Functions 94.12% ( 5604 / 5954 )
Lines 95.93% ( 17719 / 18470 )

@SahilK-027 SahilK-027 merged commit 5ca606e into master May 3, 2024
6 checks passed
Dimple16 pushed a commit that referenced this pull request May 16, 2024
Dimple16 added a commit that referenced this pull request May 16, 2024
* Revert "fix: Improved test cases ProjectsService (#2980)"

This reverts commit 77e6f4c.

* Revert "feat: Migrate getByParamsUnformatted -> spenderPlatformV1ApiService (#2954)"

This reverts commit 53f20fb.

* Revert "feat: Migrate getAllActive -> spenderPlatformV1ApiService (#2967)"

This reverts commit b5c9e2e.

* Revert "feat: Migrate getbyId -> spenderPlatformV1ApiService (#2946)"

This reverts commit e110648.

* Revert "feat: Add transformToV1Response & transformToV2Response for PlatformProject (#2942)"

This reverts commit 5ca606e.

---------

Co-authored-by: Dimple <[email protected]>
Dimple16 added a commit that referenced this pull request May 16, 2024
* Revert "fix: Improved test cases ProjectsService (#2980)"

This reverts commit 77e6f4c.

* Revert "feat: Migrate getByParamsUnformatted -> spenderPlatformV1ApiService (#2954)"

This reverts commit 53f20fb.

* Revert "feat: Migrate getAllActive -> spenderPlatformV1ApiService (#2967)"

This reverts commit b5c9e2e.

* Revert "feat: Migrate getbyId -> spenderPlatformV1ApiService (#2946)"

This reverts commit e110648.

* Revert "feat: Add transformToV1Response & transformToV2Response for PlatformProject (#2942)"

This reverts commit 5ca606e.

---------

Co-authored-by: Dimple <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/M Medium PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants