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.

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

feat(server): masquer les effectifs d'OF dont on ne connait pas la raison sociale ou l'enseigne #3356

Merged
merged 5 commits into from
Nov 10, 2023

Conversation

sbenfares
Copy link
Contributor

@sbenfares sbenfares commented Nov 10, 2023

Vu avec @nadinelouchart on masque les effectifs lié à des OF "inconnus".

  • Ajout dans le champ computed des effectifs de la raison sociale et de l'enseigne
  • Ajout dans le filtre des requêtes d'un filtrage sur ces 2 champs

Copy link

To deploy this PR just add a comment with a simple 🚀

@sbenfares sbenfares force-pushed the feature/hide-effectifs-raisonSociale-enseigne-empty branch from bfcdfe5 to bdc552f Compare November 10, 2023 14:40
Copy link
Contributor

@rap2hpoutre rap2hpoutre left a comment

Choose a reason for hiding this comment

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

Je sais pas si le test qui échoue est normal, je conseille de vérifier avant !

@sbenfares
Copy link
Contributor Author

Je sais pas si le test qui échoue est normal, je conseille de vérifier avant !

Yes je viens de voir que j'avais une coquille dans la syntaxe, c'est en cours de fix merci

@sbenfares sbenfares added this pull request to the merge queue Nov 10, 2023
Merged via the queue into master with commit 8e5d16d Nov 10, 2023
7 checks passed
@sbenfares sbenfares deleted the feature/hide-effectifs-raisonSociale-enseigne-empty branch November 10, 2023 15:48
Copy link

🎉 This PR is included in version 3.96.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

sbenfares added a commit that referenced this pull request Nov 13, 2023
…as la raison sociale ou l'enseigne (#3356)"

This reverts commit 8e5d16d.
github-merge-queue bot pushed a commit that referenced this pull request Nov 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants