Skip to content

Dev to Main Sync#215

Merged
iamitprakash merged 1 commit intomainfrom
develop
Oct 18, 2025
Merged

Dev to Main Sync#215
iamitprakash merged 1 commit intomainfrom
develop

Conversation

@AnujChhikara
Copy link
Contributor

@AnujChhikara AnujChhikara commented Oct 18, 2025

Date: 18 Oct 2025

Developer Name: @AnujChhikara

Issue Ticket Number

PRs going in Sync

Description

  • Added the poc change functionality

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Screenshots

Staging Proof
Screen.Recording.2025-10-18.at.10.50.16.PM.mov

Description by Korbit AI

What change is being made?

Add support for updating a team's POC (and other basic fields) via a new UpdateTeamParams type, a corresponding updateTeam API method, and UI flow to appoint a member as POC with success/error notifications and cache invalidations.

Why are these changes being made?

Enable selecting a new Point of Contact for a team and reflect changes in the UI and cache; streamline team updates via a single PATCH endpoint and keep frontend data in sync.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

* feat(teams): add update POC functionality with API integration

* feat(teams): invalidate tasks query on POC update for improved data consistency

* fix(teams): update API endpoint for POC update and adjust role change condition in team members component

* refactor(teams): rename updateTeamPoc to updateTeam and extend parameters for improved flexibility
@AnujChhikara AnujChhikara self-assigned this Oct 18, 2025
@coderabbitai
Copy link

coderabbitai bot commented Oct 18, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@korbit-ai korbit-ai bot left a comment

Choose a reason for hiding this comment

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

Review by Korbit AI

Korbit automatically attempts to detect when you fix issues in new commits.
Category Issue Status
Performance Unfiltered data spreading in API request ▹ view
Functionality Cannot clear POC assignment due to falsy value filtering ▹ view
Files scanned
File Path Reviewed
src/api/teams/teams.type.ts
src/api/teams/teams.api.ts
src/components/teams/team-members.tsx

Explore our documentation to understand the languages and file types we support and the files we ignore.

Check out our docs on how you can make Korbit work best for you and your team.

Loving Korbit!? Share us on LinkedIn Reddit and X

@iamitprakash iamitprakash merged commit 1d47473 into main Oct 18, 2025
4 checks passed
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