Project

General

Profile

Bug #4144

Pouvoir associer un lieu de socialisation à un ou plusieurs services

Added by Mikaël Ates almost 11 years ago. Updated about 10 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Jérôme Schneider
Target version:
1.2 Lieux de socialisation
Start date:
18 December 2013
Due date:
% Done:

100%

Estimated time:
Patch proposed:
Yes
Planning:

Description

Il faut ajouter un champs services au modèle. L'ors de l'ajout ils sont tous cochés par défaut.

Files

0001-ressources-add-services-management-for-schools.patch (52.3 KB) 0001-ressources-add-services-management-for-schools.patch Jérôme Schneider, 15 July 2014 08:17 AM
0002-ressources-add-services-management-for-schools.patch (52.3 KB) 0002-ressources-add-services-management-for-schools.patch Jérôme Schneider, 15 July 2014 10:17 AM
0001-dossiers-use-service-slug-instead-of-name.patch (1.35 KB) 0001-dossiers-use-service-slug-instead-of-name.patch Jérôme Schneider, 17 July 2014 05:18 PM
0002-ressources-add-services-management-for-schools.patch (52.6 KB) 0002-ressources-add-services-management-for-schools.patch Jérôme Schneider, 17 July 2014 05:18 PM

Associated revisions

Revision fe0dcf03 (diff)
Added by Jérôme Schneider about 10 years ago

ressources: add services management for schools

Closes #4144

History

#1

Updated by Mikaël Ates about 10 years ago

  • Target version set to 1.2 Lieux de socialisation
#2

Updated by Jérôme Schneider about 10 years ago

  • Status changed from Nouveau to En cours
  • Assignee set to Jérôme Schneider
  • Patch proposed set to No
#3

Updated by Jérôme Schneider about 10 years ago

#4

Updated by Frédéric Péters about 10 years ago

Le nom du cookie se trouve défini dans une constante, ce serait mieux de l'utiliser.

calebasse/cbv.py:HOME_SERVICE_COOKIE = 'home-service'

Le cookie est posé avec le slug comme valeur,

calebasse/cbv.py:            result.set_cookie(HOME_SERVICE_COOKIE, self.service.slug,

Ce serait du coup plus adapté de faire la recherche sur le slug, plutôt que d'essayer de reconstruire le nom via des appels à upper() et replace().

#5

Updated by Jérôme Schneider about 10 years ago

Bien vu. Nouveau patch mis à jour. En fait j'avais repris le code d'un lookups dans dossier. Je vais le corriger.

#7

Updated by Jérôme Schneider about 10 years ago

  • Status changed from En cours to Résolu (à déployer)
  • % Done changed from 80 to 100

Appliqué par commit calebasse|commit:fe0dcf03c5b84406170285778a21c35c3ee7d469.

#8

Updated by Serghei Mihai about 10 years ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF