Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • G gestioCOF
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 76
    • Issues 76
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 16
    • Merge requests 16
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • Klub Dev ENSKlub Dev ENS
  • gestioCOF
  • Merge requests
  • !401

Préparation de la vue d'autocompletion pour l'intégration du BDS

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Martin Pepin requested to merge kerl/merge_shared_utils into master Jan 02, 2020
  • Overview 14
  • Commits 5
  • Pipelines 0
  • Changes 8

Petits changements pour préparer !390 (merged)

  • Fusion des modules shared et utils : je ne sais plus comment on s'est retrouvé avec deux modules séparés mais ça ne me paraît mieux de (re)fusionner.

  • Séparation entre :

    • la recherche (classe ModelSearch)
    • et la vue django-autocomplete-light (classe Select2QuerySetView).

    L'idée et qu'on puisse garder la vue d'autocomplétion dal et réutiliser la partie "recherche" dans !390 (merged) avec un "combinateur" qui agrège les résultats de plusieurs instances de ModelSearch, les résultats d'une recherche ldap, etc. cf !390 (comment 6718).

  • Bonus : annotations de type dans shared.views.autocomplete. Je ne sais pas si on veut faire ça partout, ou à quelques endroits, ou nulle part. Je mets ça là pour montrer à quoi ça peut ressembler, je pense que dans ce genre de cas ça fait de la doc pas chère. C'est un tout petit commit, aucun souci pour le virer si on décide qu'on n'aime pas.

Edited Feb 12, 2020 by Martin Pepin
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: kerl/merge_shared_utils