Skip to content
GitLab
Projets Groupes Sujets Extraits de code
  • /
  • Aide
    • Aide
    • Support
    • Forum de la communauté
    • Proposer une rétroaction
    • Contribuer à GitLab
  • Connexion
  • M memopol
  • Informations du projet
    • Informations du projet
    • Activité
    • Étiquettes
    • Membres
  • Dépôt
    • Dépôt
    • Fichiers
    • Commits
    • Branches
    • Étiquettes
    • Statistiques sur les contributeurs
    • Graphe
    • Comparer les révisions
  • Tickets 32
    • Tickets 32
    • Liste
    • Tableaux
    • Service d’assistance
    • Jalons
  • Demandes de fusion 6
    • Demandes de fusion 6
  • Intégration et livraison continues
    • Intégration et livraison continues
    • Pipelines
    • Tâches
    • Planifications
  • Déploiements
    • Déploiements
    • Environnements
    • Versions
  • Supervision
    • Supervision
    • Incidents
  • Analytique
    • Analytique
    • Chaînes de valeur
    • Intégration et livraison continues
    • Dépôt
  • Wiki
    • Wiki
  • Extraits de code
    • Extraits de code
  • Activité
  • Graphe
  • Créer un nouveau ticket
  • Tâches
  • Commits
  • Tableaux des tickets
Masquer la barre latérale
  • La Quadrature du NetLa Quadrature du Net
  • Political MemoryPolitical Memory
  • memopol
  • Demandes de fusion
  • !85

Integrate views removed from d-r[-v]

  • Examiner les modifications

  • Télécharger
  • Correctifs
  • Diff brut
Fusionnée njoyard a demandé de fusionner representative-views-41454979 vers master juin 08, 2016
  • Vue d’ensemble 5
  • Commits 7
  • Pipelines 0
  • Modifications 40

Created by: njoyard

A quick summary of the changes made:

  • Gave names to API views because their default names conflicted with other view names
  • Added tags to make group/chamber/country URLs (1)
  • Moved views each to its own file and integrated views removed from d-r[-v]
  • Integrated URLs removed from d-r[-v]
  • Made URLs more homogenous (allow singular & plural ; all URLs default to singular) : moved /groups/ to /group/ and added 301 redirect from old to new

(1) I prefer using tags instead of using get_absolute_url. As models are defined in a dependency, they cannot "know" what the user application wants to use as the "canonical model url" (for example for groups it may be rep list filtered by this group, or a detail page for the group, or whatever). I could have made inherited models in memopol just to override get_absolute_url, but then making sure foreign keys use overriden models is not made easy AT ALL by django. It's possible, but the code is horrible.

Assigné
Assigner à
Relecteurs
Demander une revue de code à
Suivi du temps
Branche Source : representative-views-41454979