Skip to content

feat(PM-1067): list copilot applications #1068

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

Merged
merged 6 commits into from
May 9, 2025
Merged

feat(PM-1067): list copilot applications #1068

merged 6 commits into from
May 9, 2025

Conversation

hentrymartin
Copy link
Collaborator

Related JIRA Ticket:

https://topcoder.atlassian.net/browse/PM-1067

What's in this PR?

  • Added a new tab section in the copilot opportunity details page
  • Added Applications as one of the tab in it and it will be shown only to project managers or admins

Screenshot 2025-05-07 at 02 05 13

Screenshot 2025-05-07 at 02 05 23

@hentrymartin hentrymartin requested a review from kkartunov May 7, 2025 00:22
setActiveTab(activeTabHash)
}, [activeTabHash])

function handleTabChange(tabId: string): void {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please convert this to arrow function and wrap it in useCallback.

@hentrymartin
Copy link
Collaborator Author

@kkartunov I've updated the PR based on your comments, can you please have a look on it again?

Copy link
Collaborator

@kkartunov kkartunov left a comment

Choose a reason for hiding this comment

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

Looks good.

Base automatically changed from pm-578 to dev May 9, 2025 15:41
@hentrymartin hentrymartin merged commit b57e191 into dev May 9, 2025
3 checks passed
@hentrymartin hentrymartin deleted the pm-1067 branch May 9, 2025 15:41
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