Skip to content

Retire la redirection vers la demande lors d'une recherche par ID#1602

Merged
JeSuisUnCaillou merged 1 commit into
developfrom
feature/dp-1753-retirer-la-feature-qui-redirige-vers-la-demande-des-que-lon
Jun 9, 2026
Merged

Retire la redirection vers la demande lors d'une recherche par ID#1602
JeSuisUnCaillou merged 1 commit into
developfrom
feature/dp-1753-retirer-la-feature-qui-redirige-vers-la-demande-des-que-lon

Conversation

@JeSuisUnCaillou

Copy link
Copy Markdown
Collaborator

@JeSuisUnCaillou JeSuisUnCaillou requested a review from Isalafont June 4, 2026 08:32
@JeSuisUnCaillou JeSuisUnCaillou self-assigned this Jun 4, 2026
@linear

linear Bot commented Jun 4, 2026

Copy link
Copy Markdown

DP-1753

@Isalafont Isalafont left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

J'ai des suggestions notamment sur le comportement des filtres, est ce que l'ID prime sur les autres filtres ?

Comment thread app/facades/instruction/search/dashboard_search.rb Outdated
Comment thread app/facades/instruction/search/dashboard_search.rb Outdated
Comment thread spec/facades/instruction/search/dashboard_demandes_search_spec.rb Outdated
Comment thread spec/facades/instruction/search/dashboard_demandes_search_spec.rb
Comment thread spec/features/instruction/search_demandes_spec.rb
@JeSuisUnCaillou JeSuisUnCaillou force-pushed the feature/dp-1753-retirer-la-feature-qui-redirige-vers-la-demande-des-que-lon branch from 33ebc0f to f67c138 Compare June 4, 2026 11:49
@JeSuisUnCaillou JeSuisUnCaillou requested a review from Isalafont June 4, 2026 12:18
@JeSuisUnCaillou

Copy link
Copy Markdown
Collaborator Author

@Isalafont on est bon ?

@Isalafont Isalafont left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Je simplifierai juste cette methode pour être plus lisible.
Une fois fait tu pourra merge dans la foulée

Comment thread app/facades/instruction/search/dashboard_search.rb
@JeSuisUnCaillou

Copy link
Copy Markdown
Collaborator Author

Je simplifierai juste cette methode pour être plus lisible. Une fois fait tu pourra merge dans la foulée

Il faut que tu valides, si tu met "demande de changements" je peux pas merger dans la foulée.

@Isalafont

Copy link
Copy Markdown
Contributor

Je simplifierai juste cette methode pour être plus lisible. Une fois fait tu pourra merge dans la foulée

Il faut que tu valides, si tu met "demande de changements" je peux pas merger dans la foulée.

Faut fix le lint d'abord

@Isalafont Isalafont dismissed their stale review June 8, 2026 13:48

approved une fois le lint fixed

@JeSuisUnCaillou JeSuisUnCaillou force-pushed the feature/dp-1753-retirer-la-feature-qui-redirige-vers-la-demande-des-que-lon branch from 2579d3d to e1fee96 Compare June 9, 2026 08:39
La recherche par numéro affiche désormais les résultats dans la liste
au lieu de rediriger vers la fiche.
@JeSuisUnCaillou JeSuisUnCaillou force-pushed the feature/dp-1753-retirer-la-feature-qui-redirige-vers-la-demande-des-que-lon branch from e1fee96 to 959954c Compare June 9, 2026 08:55
@JeSuisUnCaillou JeSuisUnCaillou merged commit f416509 into develop Jun 9, 2026
19 checks passed
@JeSuisUnCaillou JeSuisUnCaillou deleted the feature/dp-1753-retirer-la-feature-qui-redirige-vers-la-demande-des-que-lon branch June 9, 2026 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants