Skip to content
This repository was archived by the owner on Aug 11, 2022. It is now read-only.

Admin - Flag specific user as priority #690

Closed
gobyrne opened this issue Feb 3, 2019 · 4 comments
Closed

Admin - Flag specific user as priority #690

gobyrne opened this issue Feb 3, 2019 · 4 comments
Assignees
Labels
Back-end For the cooks

Comments

@gobyrne
Copy link
Member

gobyrne commented Feb 3, 2019

Description

Admins (Talent Cloud Concierge) should have the ability to flag an applicant (Identifiable by name and/or email) as a Priority. Priorities are treated differently in the application process, so they need to be made visible to managers. The intent is to have them displayed separately on the Review Applicants page #687.

Down the road we will try and build something that allows priorities to self-identify, but for now, they have to contact Talent Cloud and our concierge service will upgrade their user profile through the admin portal.

For now users are either priorities or they are not, but in the future we might want to allow for various types (levels) of priorities. It's not of interest right now, but we might put a time limit on how long a person is a priority for (since priority status normally expires at some point).

This issue is a dependency of #690.

@gobyrne gobyrne added the Back-end For the cooks label Feb 3, 2019
@gobyrne gobyrne changed the title *Incomplete issue* - Admin portal flag priorities Admin - Flag specific user as priority Feb 3, 2019
@tristan-orourke
Copy link
Member

  • Users can be flagged as priority on the admin portal
  • Priority users appear in their own section of the Application Review page

@tristan-orourke tristan-orourke mentioned this issue Feb 22, 2019
26 tasks
@Khristophor Khristophor self-assigned this Feb 25, 2019
@Khristophor
Copy link
Contributor

@tristan-orourke I pushed up what I have working so far on feature/add-priority-to-applicants. I've hit a bit of a wall around assigning the checkbox value to the applicant priority field, as it doesn't seem to work the way I thought it could.

It seems that in the index view, you can specify a property of a relationship with dot notation, but you can't do the same for a field:
https://github.com/GCTC-NTGC/TalentCloud/blob/task/add-priority-to-applicants/app/Http/Controllers/Admin/UserCrudController.php#L77-L81

We might have to hijack the update method and set/unset the value manually.

I also grabbed the toggle control from this PR that hasn't been merged.

@Khristophor
Copy link
Contributor

After discussion with @gobyrne we opted to add a boolean field to the User model instead.
PR can be found here.

@gggrant
Copy link
Contributor

gggrant commented Mar 20, 2019

Resolved by #783

@gggrant gggrant closed this as completed Mar 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Back-end For the cooks
Projects
None yet
Development

No branches or pull requests

4 participants