-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
New Components - greenhouse #11886
New Components - greenhouse #11886
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 3 Ignored Deployments
|
Sources - Watch Candidates (Instant) - New Candidate Application - New Scheduled Interview Actions - Create Candidate - Create Prospect - Add Attachment To Candidate
@luancazarine PR is still marked as a draft - is it ready for review? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've left one suggestion to avoid code repetition, and also one package/config comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some components can still be optimized to avoid repeating code, but I'll move this forward to QA since this has no impact on functionality
components/greenhouse/actions/create-prospect/create-prospect.mjs
Outdated
Show resolved
Hide resolved
components/greenhouse/actions/create-candidate/create-candidate.mjs
Outdated
Show resolved
Hide resolved
/approve |
Resolves #11589.