Profil salarié: redonner l'accès aux anciens PASS IAE #4735
Conversation
xavfernandez
added
the
no-changelog
| @@ -57,7 +57,7 @@ <h3>{{ approval.user.get_full_name }}</h3> | |||
| <div class="c-box--results__footer"> | |||
|
|
|||
| <div class="d-flex justify-content-md-end"> | |||
| <a href="{% url 'employees:detail' public_id=approval.user.public_id %}?back_url={{ request.get_full_path|urlencode }}" | |||
| <a href="{% url 'employees:detail' public_id=approval.user.public_id %}?approval={{ approval.pk }}&back_url={{ request.get_full_path|urlencode }}" | |||
There was a problem hiding this comment.
A-t-on prévu de lister les PASS sur l’espace salarié, plutôt que d’avoir des doublons ? J’imagine qu’à terme on ira vers l’espace mes candidats (mes employés), où on listera tous les pass d’un employé, et que via la liste des PASS IAE, on ira vers la page de détail des pass ?
There was a problem hiding this comment.
Oui, j'en ai parlé à Marion & Antoine et a priori on aura plus de doublons et on aura accès à l'historique des PASS sur l'espace candidat/salarié.
| approval = None | ||
| if approval_pk := self.request.GET.get("approval"): | ||
| with contextlib.suppress(ValueError): # Ignore invalid approval parameter value | ||
| approval = Approval.objects.filter(user=self.object, pk=int(approval_pk)).first() |
There was a problem hiding this comment.
C’est un peu dommage de ne pas passer par un form pour valider ces données et de réinventer la validation d’un int qui est un PASS IAE ?
There was a problem hiding this comment.
Un form pourrait faire le travail mais ça prendrait sûrement plus de 2 lignes avec une indirection en plus 😬
Sachant que ce hack est normalement temporaire cela me semble good enough .
itou/www/employees_views/views.py
Outdated
| # This shouldn't be possible except if the job application has been deleted | ||
| # in this case, use the last approval |
There was a problem hiding this comment.
Un employer peut accéder à cette page via le public id de n’importe quel candidat, même si ce candidat n’a pas de candidatures ni de pass. Le commentaire est incomplet, ce que je trouve déroutant.
There was a problem hiding this comment.
Oui j'ai rajouté une règle d'accès plus claire pour la page.
Les règles sur quel PASS/Candidature/Diag est utilisé dans cette page me semble un peu confus mais on revient a priori sur le comportement actuel et on nettoiera tout cela à la prochaine refonte de cette page.
xavfernandez
force-pushed
the
xfernandez/allow_to_show_old_approval
branch
from
d3fff33 to
3a0a1e7
Compare
xavfernandez
force-pushed
the
xfernandez/allow_to_show_old_approval
branch
from
3a0a1e7 to
33d0df9
Compare
🤔 Pourquoi ?
Suite à #4713 l'accès au Profile salarié se fait via l'identifiant public du compte candidat.
Or certains candidats ont plusieurs PASS et apparaissent plusieurs fois dans la liste mais avec toujours le même lien menant au profil salarié affichant le dernier PASS (depuis #4731) et avec impossibilité de revoir les infos des anciens PASS.
Cette PR corrige ce soucis.
🍰 Comment ?
🚨 À vérifier
🏝️ Comment tester
💻 Captures d'écran