New work app to replace work-manager, and v6 projects API usage in copilots app #1487
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| REACT_APP_GROUPS_API_URL=https://api.topcoder-dev.com/v6/groups | ||
| REACT_APP_TERMS_API_URL=https://api.topcoder-dev.com/v5/terms | ||
| REACT_APP_RESOURCES_API_URL=https://api.topcoder-dev.com/v6/resources | ||
| REACT_APP_MEMBER_API_URL=https://api.topcoder-dev.com/v6/members | ||
| REACT_APP_RESOURCE_ROLES_API_URL=https://api.topcoder-dev.com/v6/resource-roles |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| REACT_APP_GROUPS_API_URL=https://api.topcoder.com/v6/groups | ||
| REACT_APP_TERMS_API_URL=https://api.topcoder.com/v5/terms | ||
| REACT_APP_RESOURCES_API_URL=https://api.topcoder.com/v6/resources | ||
| REACT_APP_MEMBER_API_URL=https://api.topcoder.com/v6/members | ||
| REACT_APP_RESOURCE_ROLES_API_URL=https://api.topcoder.com/v6/resource-roles | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,18 @@ | ||
| import { shouldUseUpdateTraitAction } from './trait-action.utils' | ||
|
|
||
| describe('shouldUseUpdateTraitAction', () => { | ||
| it('returns true when the initial trait exists', () => { | ||
| expect(shouldUseUpdateTraitAction({ traitId: 'software' }, undefined)) | ||
| .toBe(true) | ||
| }) | ||
|
|
||
| it('returns true when local traits exist even without the initial trait', () => { | ||
| expect(shouldUseUpdateTraitAction(undefined, [{ name: 'Chrome' }])) | ||
| .toBe(true) | ||
| }) | ||
|
|
||
| it('returns false when both initial and local traits are missing', () => { | ||
| expect(shouldUseUpdateTraitAction(undefined, undefined)) | ||
| .toBe(false) | ||
| }) | ||
| }) |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| import { UserTrait } from '~/libs/core' | ||
|
|
||
| /** | ||
| * Determine whether tool traits should use update or create action. | ||
| * The initial trait prop can be stale while the user stays on the tab, | ||
| * so local list state is also considered to avoid duplicate create calls. | ||
| */ | ||
| export function shouldUseUpdateTraitAction( | ||
| initialTrait: UserTrait | undefined, | ||
| localTraitsData: UserTrait[] | undefined, | ||
| ): boolean { | ||
| return Boolean(initialTrait || localTraitsData?.length) | ||
|
There was a problem hiding this comment. [ |
||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,30 @@ | ||
| import { | ||
| canOpenReviewUi, | ||
| getReviewUiChallengeUrl, | ||
| } from './reviewUiLink' | ||
|
|
||
| describe('ChallengeList review UI helpers', () => { | ||
| describe('canOpenReviewUi', () => { | ||
| it('returns true when challenge has a uuid id', () => { | ||
| expect(canOpenReviewUi('challenge-uuid')) | ||
| .toBe(true) | ||
| }) | ||
|
|
||
| it('returns false when challenge id is empty', () => { | ||
| expect(canOpenReviewUi('')) | ||
| .toBe(false) | ||
| }) | ||
|
|
||
| it('returns false when challenge id is only whitespace', () => { | ||
| expect(canOpenReviewUi(' ')) | ||
| .toBe(false) | ||
| }) | ||
| }) | ||
|
|
||
| describe('getReviewUiChallengeUrl', () => { | ||
| it('builds review ui url using challenge id path', () => { | ||
| expect(getReviewUiChallengeUrl('https://review.topcoder-dev.com', 'challenge-uuid')) | ||
| .toBe('https://review.topcoder-dev.com/challenge-uuid') | ||
| }) | ||
| }) | ||
| }) |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| /** | ||
| * Returns whether the Review UI link can be opened for a challenge. | ||
| */ | ||
| export function canOpenReviewUi(challengeId?: string): boolean { | ||
| return Boolean(challengeId?.trim()) | ||
|
There was a problem hiding this comment. [💡 |
||
| } | ||
|
|
||
| /** | ||
| * Builds the Review UI URL for a challenge id. | ||
| */ | ||
| export function getReviewUiChallengeUrl( | ||
| reviewUiBaseUrl: string, | ||
| challengeId: string, | ||
| ): string { | ||
| return `${reviewUiBaseUrl}/${challengeId}` | ||
|
There was a problem hiding this comment. [ |
||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,89 @@ | ||
| @import '@libs/ui/styles/includes'; | ||
|
|
||
| .modal { | ||
| width: 1240px !important; | ||
|
There was a problem hiding this comment. [ |
||
| max-width: calc(100vw - 32px) !important; | ||
|
There was a problem hiding this comment. [ |
||
| } | ||
|
|
||
| .container { | ||
| display: flex; | ||
| flex-direction: column; | ||
| gap: 20px; | ||
|
|
||
| th:first-child { | ||
| padding-left: 16px !important; | ||
|
There was a problem hiding this comment. [ |
||
| } | ||
| } | ||
|
|
||
| .actionButtons { | ||
| display: flex; | ||
| justify-content: flex-end; | ||
| gap: 6px; | ||
| } | ||
|
|
||
| .tableCellNoWrap { | ||
| white-space: nowrap; | ||
| text-align: left !important; | ||
|
There was a problem hiding this comment. [ |
||
| } | ||
|
|
||
| .statusCell { | ||
| text-align: center !important; | ||
|
There was a problem hiding this comment. [ |
||
| width: 90px; | ||
| } | ||
|
|
||
| .emailStatus { | ||
| align-items: center; | ||
| display: inline-flex; | ||
| justify-content: center; | ||
| min-height: 24px; | ||
| min-width: 24px; | ||
|
|
||
| svg { | ||
| width: 20px; | ||
| height: 20px; | ||
| } | ||
| } | ||
|
|
||
| .emailStatusDelivered { | ||
| color: $green-120; | ||
| } | ||
|
|
||
| .emailStatusFailed { | ||
| color: $red-110; | ||
| } | ||
|
|
||
| .tableCell { | ||
| min-width: 220px; | ||
| white-space: break-spaces !important; | ||
| text-align: left !important; | ||
|
There was a problem hiding this comment. [ |
||
| } | ||
|
|
||
| .loadingSpinnerContainer { | ||
| position: relative; | ||
| height: 100px; | ||
|
|
||
| .spinner { | ||
| background: none; | ||
| } | ||
| } | ||
|
|
||
| .noRecordFound { | ||
| padding: 16px 16px 32px; | ||
| text-align: center; | ||
| } | ||
|
|
||
| .desktopTable { | ||
| overflow-x: auto; | ||
| overflow-y: visible; | ||
|
There was a problem hiding this comment. [ |
||
|
|
||
| thead th { | ||
| position: sticky; | ||
| top: 0; | ||
| z-index: 2; | ||
| background: $tc-white; | ||
| } | ||
|
|
||
| td { | ||
| vertical-align: middle; | ||
| } | ||
| } | ||
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.
[❗❗
correctness]The API version for
REACT_APP_TERMS_API_URLis v5, while others are v6. Ensure this is intentional and that the correct API version is being used.