Skip to content

Candidature: ajout d'un lien vers la fiche structure pour les utilisateurs n'appartenant pas à la structure [GEN-2234]#5131

Merged
xavfernandez merged 1 commit intomasterfrom
xfernandez/add_company_link
Nov 21, 2024
Merged

Candidature: ajout d'un lien vers la fiche structure pour les utilisateurs n'appartenant pas à la structure [GEN-2234]#5131
xavfernandez merged 1 commit intomasterfrom
xfernandez/add_company_link

Conversation

@xavfernandez
Copy link
Contributor

🤔 Pourquoi ?

Car ce lien n'existait pas.

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@xavfernandez xavfernandez added the ajouté Ajouté dans le changelog. label Nov 20, 2024
@xavfernandez xavfernandez self-assigned this Nov 20, 2024
@xavfernandez xavfernandez added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Nov 20, 2024
@github-actions
Copy link

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@xavfernandez xavfernandez force-pushed the xfernandez/add_company_link branch 2 times, most recently from ba8a7fa to 1dd4283 Compare November 20, 2024 16:26
@xavfernandez xavfernandez force-pushed the xfernandez/add_company_link branch from 1dd4283 to 42fffb5 Compare November 20, 2024 16:41
<li>
<small>Employeur destinataire</small>
<strong>{{ job_application.to_company.display_name }}</strong>
<a class="btn btn-link" href="{% url 'companies_views:card' siae_id=job_application.to_company.id %}?back_url={{ request.get_full_path|urlencode }}"><strong>{{ job_application.to_company.display_name }}</strong></a>
Copy link
Contributor

Choose a reason for hiding this comment

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

Je me demande si faudrait pas ouvrir dans un nouvel onglet (et ajouter le picto qui va bien) 🤔, je connais pas trop la règle qui permet de décider ça.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

C'était ce qui était prévu dans la carte 👍
Mais actuellement les liens des fiches de postes juste en dessous ne s'ouvrent pas dans un nouvel onglet, donc pour être cohérent on laisse en l'état et on changera tout d'un coup.

@xavfernandez xavfernandez added this pull request to the merge queue Nov 21, 2024
Merged via the queue into master with commit fabb1d1 Nov 21, 2024
@xavfernandez xavfernandez deleted the xfernandez/add_company_link branch November 21, 2024 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC ajouté Ajouté dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants