Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Retire le champ de recherche (inactif) sur les profils #6201

Merged
merged 2 commits into from
Oct 31, 2021
Merged

Retire le champ de recherche (inactif) sur les profils #6201

merged 2 commits into from
Oct 31, 2021

Conversation

Arnaud-D
Copy link
Contributor

Fix #6191.

Le champ de recheche n'a pas été rendu fonctionnel depuis des mois ou des années. On pourra le remettre quand on aura quelque chose à proposer. ^^

Contrôle qualité

  • vérifier que ça ne casse pas le design sur écran normal
  • pareil sur mobile

@Arnaud-D Arnaud-D added the C-Front Concerne l'interface du site label Oct 29, 2021
@Arnaud-D Arnaud-D added this to En développement in Suivi des PR via automation Oct 29, 2021
@Arnaud-D Arnaud-D moved this from En développement to En attente de QA in Suivi des PR Oct 29, 2021
@coveralls
Copy link

coveralls commented Oct 29, 2021 •
edited
Loading

Coverage Status

Coverage remained the same at 86.732% when pulling fa2949e on Arnaud-D:fini_avant_d'avoir_commencé into 9caed20 on zestedesavoir:dev.

@Arnaud-D Arnaud-D added the hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest label Oct 31, 2021
philippemilink
philippemilink approved these changes Oct 31, 2021
View reviewed changes
Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

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

Rapport de QA

OK ✔️

Testé en étant connecté ou non, sur Firefox et Safari, avec la vue adaptative pour tester différentes résolutions.

Suivi des PR automation moved this from En attente de QA to Fusionnable après rebase Oct 31, 2021
@philippemilink philippemilink enabled auto-merge (squash) October 31, 2021 19:40
@philippemilink philippemilink merged commit f3080ee into zestedesavoir:dev Oct 31, 2021
Suivi des PR automation moved this from Fusionnable après rebase to Fusionnée Oct 31, 2021
@Arnaud-D Arnaud-D deleted the fini_avant_d'avoir_commencé branch October 31, 2021 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@philippemilink philippemilink philippemilink approved these changes

Assignees
No one assigned
Labels
C-Front Concerne l'interface du site hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
Suivi des PR
Archived in project
Suivi des PR
  
Fusionnée
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Barre de recherche dans un profil auteur
3 participants
@Arnaud-D @coveralls @philippemilink