GEIQ : Masquer les diagnostics d’éligibilité employeurs aux prescripteurs [GEN-1975]#4783
Merged
francoisfreitag merged 1 commit intomasterfrom Oct 1, 2024
Merged
GEIQ : Masquer les diagnostics d’éligibilité employeurs aux prescripteurs [GEN-1975]#4783francoisfreitag merged 1 commit intomasterfrom
francoisfreitag merged 1 commit intomasterfrom
Conversation
Previously, prescribers would see diagnosis created by the GEIQ. - if they decided not to update the diag, the job seeker would not get an eligibility diag from an authorized prescriber, that allows companies hiring them to get the max subsidy. That would hinder the job seeker. - else they decided to update the diagnosis, the system would update the existing **GEIQ** diag instead of creating a new diag, resulting in a diagnosis of GEIQ kind, with a prescriber as its author.
francoisfreitag
added
the
1-recette-jetable
francoisfreitag
changed the title
francoisfreitag
requested review from
tonial and
xavfernandez
and removed request for
xavfernandez
tonial
approved these changes
Sep 20, 2024
francoisfreitag
removed
the
1-recette-jetable
| if self.company.kind == CompanyKind.GEIQ: | ||
| self.geiq_eligibility_diagnosis = GEIQEligibilityDiagnosis.objects.valid_diagnoses_for( | ||
| self.job_seeker, self.company | ||
| self.job_seeker, self.company if self.request.user.is_employer else None |
Contributor
There was a problem hiding this comment.
Est-ce qu'on ne voudrait pas juste le cacher pour les prescripteurs habilités ?
Là un orienteur n'aura pas de diag GEIQ dans son parcours (et ne pourra pas en créer un).
Member
Author
There was a problem hiding this comment.
Pour moi non, l’orienteur n’a pas à savoir que l’employeur a établi un diagnostic d’éligibilité, ça ne le regarde pas.
Aussi, il est facile de se méprendre sur le fait qu’un candidat soit éligible à une aide grâce à un diag employeur, qui ne sera pas valide chez un autre employeur. Pour moi, le diag d’un employeur n’est visible que par l’employeur qui l’a établi.
xavfernandez
approved these changes
Sep 26, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤔 Pourquoi ?
Auparavant, les prescripteurs voyaient les diagnostics d’éligibilité établis par les employeurs lors du processus de prescription.
🏝️ Comment tester