Skip to content

[PROD RELEASE] #1653

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

Merged
merged 21 commits into from
Jun 18, 2025
Merged

[PROD RELEASE] #1653

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
4ec89b1
fix: pagination in submissions tab
hentrymartin May 21, 2025
6b81b05
fix: pagination in submissions tab
hentrymartin May 21, 2025
cd5322c
fix: pagination in submissions tab
hentrymartin May 21, 2025
1e3c8e0
Merge pull request #1649 from topcoder-platform/pm-1214
kkartunov May 22, 2025
0eb4fb3
fix: send source to accept or decline invite API
hentrymartin May 30, 2025
8390251
fix: send source to accept or decline invite API
hentrymartin May 30, 2025
713528f
fix: send source to accept or decline invite API
hentrymartin May 30, 2025
a211634
Merge pull request #1650 from topcoder-platform/pm-1169
kkartunov Jun 3, 2025
6063bbe
PM-1178 Update copilot requests form link
Jun 4, 2025
f489bae
PM-1178 Clean up typeform code
Jun 4, 2025
0b819d7
Merge pull request #1651 from topcoder-platform/PM-1178
kkartunov Jun 5, 2025
fc93578
fix: show request copilot for PMs
hentrymartin Jun 10, 2025
9c67f2b
Merge pull request #1652 from topcoder-platform/pm-1317
hentrymartin Jun 10, 2025
3b67a7e
fix: retain query param
hentrymartin Jun 16, 2025
153ab96
fix: retain query param
hentrymartin Jun 16, 2025
1aa70f4
fix: retain query param
hentrymartin Jun 16, 2025
78755ed
Merge pull request #1654 from topcoder-platform/pm-1355
hentrymartin Jun 16, 2025
b3eb22c
fix: send source only if its available
hentrymartin Jun 16, 2025
612a798
fix: send source only if its available
hentrymartin Jun 16, 2025
8d4d3a8
fix: send source only if its available
hentrymartin Jun 16, 2025
37c544a
Merge pull request #1655 from topcoder-platform/pm-1355_1
hentrymartin Jun 16, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
PM-1178 Update copilot requests form link
  • Loading branch information
himaniraghav3 committed Jun 4, 2025
commit 6063bbe816852e1511b454527b1d7ce84a76059d
1 change: 1 addition & 0 deletions config/constants/development.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ module.exports = {
CHALLENGE_TRACKS_URL: `${DEV_API_HOSTNAME}/v5/challenge-tracks`,
CHALLENGE_PHASES_URL: `${DEV_API_HOSTNAME}/v5/challenge-phases`,
CHALLENGE_TIMELINES_URL: `${DEV_API_HOSTNAME}/v5/challenge-timelines`,
COPILOTS_URL: `https://copilots.${DOMAIN}`,
Copy link

Choose a reason for hiding this comment

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

Consider using the DEV_API_HOSTNAME variable for consistency with other URLs, unless DOMAIN is intended to be different for this specific endpoint.

PROJECT_API_URL: `${DEV_API_HOSTNAME}/v5/projects`,
GROUPS_API_URL: `${DEV_API_HOSTNAME}/v5/groups`,
TERMS_API_URL: `${DEV_API_HOSTNAME}/v5/terms`,
Expand Down
1 change: 1 addition & 0 deletions config/constants/production.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ module.exports = {
CHALLENGE_TRACKS_URL: `${PROD_API_HOSTNAME}/v5/challenge-tracks`,
CHALLENGE_PHASES_URL: `${PROD_API_HOSTNAME}/v5/challenge-phases`,
CHALLENGE_TIMELINES_URL: `${PROD_API_HOSTNAME}/v5/challenge-timelines`,
COPILOTS_URL: `https://copilots.${DOMAIN}`,
Copy link

Choose a reason for hiding this comment

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

The COPILOTS_URL is using DOMAIN directly, but other URLs are using PROD_API_HOSTNAME. Consider using PROD_API_HOSTNAME for consistency, or ensure DOMAIN is correctly defined and intended for use here.

PROJECT_API_URL: `${PROD_API_HOSTNAME}/v5/projects`,
GROUPS_API_URL: `${PROD_API_HOSTNAME}/v5/groups`,
TERMS_API_URL: `${PROD_API_HOSTNAME}/v5/terms`,
Expand Down
4 changes: 2 additions & 2 deletions src/components/ChallengesComponent/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ import PropTypes from 'prop-types'
import { Helmet } from 'react-helmet'
import { Link } from 'react-router-dom'
import ProjectStatus from './ProjectStatus'
import { PROJECT_ROLES, TYPEFORM_URL, PROJECT_STATUS } from '../../config/constants'
import { PROJECT_ROLES, PROJECT_STATUS, COPILOTS_URL } from '../../config/constants'
import { PrimaryButton, OutlineButton } from '../Buttons'
import ChallengeList from './ChallengeList'
import styles from './ChallengesComponent.module.scss'
Expand Down Expand Up @@ -98,7 +98,7 @@ const ChallengesComponent = ({
<OutlineButton
text='Request Copilot'
type={'info'}
url={`${TYPEFORM_URL}#handle=${auth.user.handle}&projectid=${activeProjectId}`}
url={`${COPILOTS_URL}/requests/new`}
Copy link

Choose a reason for hiding this comment

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

Ensure that COPILOTS_URL is defined and correctly configured to point to the intended endpoint. This change modifies the URL structure, so verify that the new URL meets the requirements.

target={'_blank'}
/>
)}
Expand Down
1 change: 1 addition & 0 deletions src/config/constants.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
export const {
COMMUNITY_APP_URL,
CHALLENGE_API_URL,
COPILOTS_URL,
SUBMISSION_REVIEW_APP_URL,
STUDIO_URL,
CONNECT_APP_URL,
Expand Down