Skip to content

feat(mentor):implement opportunities management - #524

Open
Arya3077 wants to merge 6 commits into
gtech-mulearn:devfrom
Arya3077:mentor-1
Open

Arya3077 wants to merge 6 commits into
gtech-mulearn:devfrom
Arya3077:mentor-1

Conversation

@Arya3077

Copy link
Copy Markdown
Contributor

No description provided.

@vercel

vercel Bot commented Sep 21, 2026

Copy link
Copy Markdown

@Arya3077 is attempting to deploy a commit to the awindsr's projects Team on Vercel.

A member of the Team first needs to authorize it.

@netlify

netlify Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

👷 Deploy request for staging-mulearn pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit e71094c

@greptile-apps

greptile-apps Bot commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Medium risk] Implements opportunities management feature for mentors.

The PR appears safe to merge on the reviewed changes, with one non-blocking previous import-boundary finding still open.

Findings

  1. P2 Onboarding Internals Imported ▶

Summary

The PR restores mentor opportunity management with list, detail, create, edit, publish, close, and archive flows, and adds a public opportunities list. Since the previous review, it sends status filters to the list endpoint and renders the returned results directly in each status tab.

Reviews (4) · Last reviewed commit: "Merge branch 'mentor-1' of https://githu..."

SelectValue,
} from "@/components/ui/select";
import { Textarea } from "@/components/ui/textarea";

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 Onboarding Internals Imported

The new organization lookup imports useCompanies through the onboarding feature's internal hooks directory instead of its public barrel. This couples the opportunities feature to onboarding's internal file structure, so an internal reorganization can break this feature. Export the hook from @/features/onboarding and import it through that public entry point.

This branch has not been deployed

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

Labels

None yet

Projects

Status: Triage
Status: Backlog

Development

Successfully merging this pull request may close these issues.

1 participant