Skip to content

Conversation

@jmgasper
Copy link
Collaborator

@jmgasper jmgasper commented Nov 2, 2025

No description provided.

@jmgasper jmgasper merged commit 2dc7055 into master Nov 2, 2025
6 checks passed
CHALLENGE_PHASES_URL: `${PROD_API_HOSTNAME}/v6/challenge-phases`,
CHALLENGE_TIMELINES_URL: `${PROD_API_HOSTNAME}/v6/challenge-timelines`,
COPILOTS_URL: `https://copilots.${DOMAIN}`,
PROJECT_API_URL: `${PROD_API_HOSTNAME}/v5/projects`,
Copy link

Choose a reason for hiding this comment

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

[⚠️ correctness]
The PROJECT_API_URL is still using version v5, while other URLs have been updated to v6. Ensure this is intentional and verify if it should be updated to maintain consistency across API versions.

PROJECT_API_URL: `${PROD_API_HOSTNAME}/v5/projects`,
GROUPS_API_URL: `${PROD_API_HOSTNAME}/v5/groups`,
GROUPS_API_URL: `${PROD_API_HOSTNAME}/v6/groups`,
TERMS_API_URL: `${PROD_API_HOSTNAME}/v5/terms`,
Copy link

Choose a reason for hiding this comment

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

[⚠️ correctness]
The TERMS_API_URL is still using version v5, while other URLs have been updated to v6. Confirm if this is intentional or if it should be updated for consistency.

GROUPS_API_URL: `${PROD_API_HOSTNAME}/v5/groups`,
GROUPS_API_URL: `${PROD_API_HOSTNAME}/v6/groups`,
TERMS_API_URL: `${PROD_API_HOSTNAME}/v5/terms`,
MEMBERS_API_URL: `${PROD_API_HOSTNAME}/v5/members`,
Copy link

Choose a reason for hiding this comment

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

[⚠️ correctness]
The MEMBERS_API_URL is still using version v5, while other URLs have been updated to v6. Verify if this is intentional or if it should be updated for consistency.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants