Skip to content

feat(onboarding): add maintainer queue settings panel (fixes #679)#746

Open
Diwakar-odds wants to merge 2 commits into
Coder-s-OG-s:mainfrom
Diwakar-odds:feat/issue-679-queue-settings
Open

feat(onboarding): add maintainer queue settings panel (fixes #679)#746
Diwakar-odds wants to merge 2 commits into
Coder-s-OG-s:mainfrom
Diwakar-odds:feat/issue-679-queue-settings

Conversation

@Diwakar-odds

Copy link
Copy Markdown

Summary

Adds the Queue Settings UI panel for the maintainer onboarding page, completing the second half of step 2.

Type of Change

  • Bug fix
  • New feature
  • UI / UX improvement
  • Refactor
  • Documentation
  • Other

Related Issue

Closes #679

What was changed?

  • Created <QueueSettings /> client component to render the three core queue controls: Minimum Contributor Level, AI-Generated PR Detection, and Auto-Assign Mentor Chain.
  • Integrated optimistic UI updates wrapping the existing maintainer server actions.
  • Modified page.tsx in src/app/onboarding/repos to fetch getInstallationSettings and display both the RepoPicker and QueueSettings side-by-side using a responsive grid/flex layout.
  • Removed max-w-2xl from the RepoPicker so it dynamically fits the two-column layout.

Screenshots

N/A

Checklist

  • My code follows the project structure and conventions
  • I tested this locally (npm run dev)
  • No hardcoded secrets or credentials
  • I have updated documentation if needed

@vercel

vercel Bot commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

@Diwakar-odds is attempting to deploy a commit to the codersogs-3057's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Diwakar-odds

Copy link
Copy Markdown
Author

Hi maintainers, could I please get a review on this PR?

ECSoC26 Label Request:
Based on the official ECSoC criteria, I believe this PR qualifies for Level 2 and good-ui.

  • Level 2 Justification: This PR introduces a new user interface component (<QueueSettings />) for the maintainer onboarding flow, integrating it directly with existing backend server actions. It resolves the "two halves" design described in the issue.
  • good-ui Justification: The implementation utilizes optimistic UI updates with React's useTransition hook to ensure a snappy user experience when toggling settings, and seamlessly integrates the new panel alongside the existing RepoPicker with a responsive two-column flex layout that matches the application's aesthetic.

Could you please add the ECSoC26, Level 2, and good-ui labels to this PR? Thank you!

@pavsoss

pavsoss commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

@Diwakar-odds the CI check is failing

@Soumya-codr

Copy link
Copy Markdown
Collaborator

@Diwakar-odds the ci checks are failing are u still active to respond

@Diwakar-odds

Copy link
Copy Markdown
Author

I have fixed the TypeScript error by casting the parsed integer to the expected literal type.

@Ayush4958

Copy link
Copy Markdown
Collaborator

@Diwakar-odds
could u add Screen recording of ur change,
it helps us to review it

thanks...

@Ayush4958 Ayush4958 added the Needs author reply Author need to reply label Jul 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Maintainer onboarding step 2: repo picker + queue settings

4 participants