Skip to content

[IMP] Recruitment: Update schedule interviews #13690

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

Closed

Conversation

larm-odoo
Copy link
Contributor

Updating to new standards

Task card for this PR.

@larm-odoo larm-odoo self-assigned this Jun 12, 2025
@larm-odoo larm-odoo added the 2 label Jun 12, 2025
@robodoo
Copy link
Collaborator

robodoo commented Jun 12, 2025

Pull request status dashboard

@larm-odoo larm-odoo marked this pull request as draft June 12, 2025 21:51
@larm-odoo larm-odoo force-pushed the 18.0-recruitment-update-schedule-interviews-larm branch from fd1470c to d444889 Compare June 23, 2025 20:40
@larm-odoo larm-odoo marked this pull request as ready for review June 23, 2025 20:40
@larm-odoo larm-odoo requested a review from slinkous June 23, 2025 20:41
@larm-odoo larm-odoo added 3 and removed 2 labels Jun 23, 2025
@C3POdoo C3POdoo requested a review from a team June 23, 2025 20:41
@larm-odoo
Copy link
Contributor Author

Hi @slinkous - this is ready for a peer review!

@larm-odoo larm-odoo force-pushed the 18.0-recruitment-update-schedule-interviews-larm branch from d444889 to 40d9a19 Compare June 27, 2025 21:22
Copy link
Contributor

@Felicious Felicious left a comment

Choose a reason for hiding this comment

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

The update works as expected, @larm-odoo !

I left some content improvement suggestions:

  • “Applicant scheduled interviews” subsection can be tighter.
  • Move the benefit statement to the top so readers see the value first.
  • Replace the detailed click-path with a short sentence and a cross-reference to the generic “Modify Stage” doc.
  • Result: a shorter, value-focused section that’s easier for customers to scan.

I'll take a quick look after the updates and i think it'll be a quick approval (:

@larm-odoo larm-odoo force-pushed the 18.0-recruitment-update-schedule-interviews-larm branch from 40d9a19 to 7c85aa7 Compare June 27, 2025 23:20
@larm-odoo larm-odoo requested a review from Felicious June 27, 2025 23:25
Copy link
Contributor

@Felicious Felicious left a comment

Choose a reason for hiding this comment

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

Looks good to me, @larm-odoo ! Had some optional suggestions, and after you address them, this PR is ready to go!

@robodoo delegate+

@larm-odoo larm-odoo force-pushed the 18.0-recruitment-update-schedule-interviews-larm branch from 7c85aa7 to e0a3923 Compare July 2, 2025 14:52
@larm-odoo
Copy link
Contributor Author

@robodoo r+

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.

3 participants