Skip to content

Production deployment - Apr 25#3823

Merged
rathorevaibhav merged 21 commits intomainfrom
develop
Apr 25, 2026
Merged

Production deployment - Apr 25#3823
rathorevaibhav merged 21 commits intomainfrom
develop

Conversation

@rathorevaibhav
Copy link
Copy Markdown
Member

@rathorevaibhav rathorevaibhav commented Apr 25, 2026

Summary

Production deployment for Apr 25. Bundles the changes merged into develop since the previous deploy.

Included changes

Features

Bug fixes

  • #3814 Fix opportunity status field save and display bug #3815 Fix opportunity status field save and display:
    • Correct Closedclosed mapping in opportunities-status config and add the missing closeddraft entry to opportunities-status-wp-mapping.
    • Compare status db value (not display label) in edit form dropdown.
    • Restore selected status on create form after validation failure.
    • Feature tests covering config integrity, status persistence, and edit form rendering.

Reverts

Chore

  • Update branch references from master/release to develop/main in CLAUDE.md and the implementation-executor agent.

Test plan

  • Verify GitHub issue creation flow uses the new YAML forms (bug, feature, task) and blank issues are disabled.
  • Job filter dropdown on the applications page shows only published jobs, sorted alphabetically.
  • Opportunity status save/edit/create flows persist and re-select the correct status, including Closed.
  • Applicant Details and Evaluate pages no longer render the "Copy for AI evaluation" button.
  • Smoke-test core HR/recruitment flows post-deploy.

rathorevaibhav and others added 18 commits April 14, 2026 11:25
Replace .md issue templates with .yml issue forms for structured fields,
dropdowns, validation, and a better issue-filing experience. Add config.yml
to disable blank issues.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…tes-to-yml

Convert issue templates from Markdown to YAML form format
The 'Closed' key in opportunities-status was incorrectly mapped to
'draft' instead of 'closed', causing the Closed status to silently
save as Draft. Also adds the missing 'closed' => 'draft' entry to
opportunities-status-wp-mapping to prevent a runtime undefined index
error in JobObserver when status is 'closed'.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The @foreach loop iterates config as $status (display label) =>
$label (db value). The selected comparison was incorrectly using
$status (display label) instead of $label (db value), so the current
saved status was never shown as selected in the dropdown.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
When the create form re-renders after a validation error, old('status')
holds the db value (e.g. 'published') but was being compared against
the display label (e.g. 'Published'), so the previously selected
status was lost. Changed comparison to use $label (the db value).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Tests cover config integrity (no duplicate db values, every status
value has a WP mapping, Closed maps to 'closed'), status persistence
on update for all four valid status values, and edit form rendering
with the correct option selected.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
CLAUDE.md and the implementation-executor agent still referenced the old
branch names (master/release), causing agents to create branches from
master and open PRs targeting master instead of develop.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Assembles clipboard text (name, applied date, form fields) and renders
a btn-clipboard span that reuses the existing ClipboardJS global setup.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Includes the partial in the card header action area so users can
copy formatted candidate details to clipboard in one click.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Includes the partial in the Applicant Details card header for both
the pre-trial and regular rounds Resume Screening sections.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…status-field

#3814 Fix opportunity status field save and display bug
…-dropdown

#3813 Filter and sort job filter dropdown to show only published jobs
…details-for-ai-evaluation

#3816 Copy candidate details for AI evaluation
@rathorevaibhav rathorevaibhav self-assigned this Apr 25, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 25, 2026

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.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 827ffe64-f3dc-4440-ba99-278646c7c331

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

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • 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.

@cypress
Copy link
Copy Markdown

cypress Bot commented Apr 25, 2026

Employee portal    Run #9035

Run Properties:  status check passed Passed #9035  •  git commit f774d31362: ci: skip coverage comment on push events
Project Employee portal
Branch Review develop
Run status status check passed Passed #9035
Run duration 00m 22s
Commit git commit f774d31362: ci: skip coverage comment on push events
Committer Vaibhav Rathore
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 4
View all changes introduced in this branch ↗︎

rathorevaibhav and others added 2 commits April 25, 2026 12:16
Removes the "Copy for AI evaluation" button from the applicant details
and evaluate pages, and deletes the shared blade partial introduced in
#3818.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…valuation

Remove Copy for AI evaluation feature
The lucassabreu/comment-coverage-clover action requires a pull request
context to post a comment. Guard the step with an event check so it
only runs on pull_request triggers, preventing failures on push runs.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@rathorevaibhav rathorevaibhav merged commit a13b23e into main Apr 25, 2026
10 of 13 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