-
Notifications
You must be signed in to change notification settings - Fork 24
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
PSP-7075 : Show ACQ team (organization) in project exports #3568
Conversation
✅ No secrets were detected in the code. |
Codecov Report
@@ Coverage Diff @@
## dev #3568 +/- ##
==========================================
- Coverage 69.52% 69.52% -0.01%
==========================================
Files 1369 1369
Lines 33661 33687 +26
Branches 6260 6268 +8
==========================================
+ Hits 23404 23420 +16
- Misses 10005 10015 +10
Partials 252 252
Flags with carried forward coverage won't be shown. Click here to find out more.
|
✅ No secrets were detected in the code. |
@@ -19,3 +22,22 @@ export enum ProjectExportTypes { | |||
COMPENSATION = 'Compensation Requisition Export', | |||
AGREEMENT = 'Agreement Export', | |||
} | |||
|
|||
const getParameterIdFromOptions = ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would you mind moving this out to a shared method (since I think your last pr uses this as well). maybe in contactutils?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
different logic, this one returns an array, the other one returns a string. I thought about it.
✅ No secrets were detected in the code. |
No description provided.