Projet

Général

Profil

Bug #5014

Ajouter un dialog de recherche du lieu de socialisation

Ajouté par Mikaël Ates (de retour le 29 avril) il y a presque 10 ans. Mis à jour il y a plus de 9 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Jérôme Schneider
Version cible:
1.2 Lieux de socialisation
Début:
20 juin 2014
Echéance:
% réalisé:

90%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Pour l'ajout d'un lieu de socialisation ajouter une première fenêtre de recherche du lieu dans laquelle on choisi le type, si public/privé, la ville ou le code postal, puis bouton rechercher. Cela affiche une liste de lieux. Après sélection du lieux est affiché la fenêtre pour les autres champs d'ajout de la période de socialisation.

Fichiers

0001-ressources-improved-socialisation-place-search.patch (27,2 ko) 0001-ressources-improved-socialisation-place-search.patch Jérôme Schneider, 10 juillet 2014 18:25

Historique

#1

Mis à jour par Jérôme Schneider il y a presque 10 ans

  • Patch proposed mis à Non

Petite proposition un peu différente mais qui me semble plus pratique :

  • élargir le champ lieu de socialisation
  • rajouter une case public / privée
  • gérer la recherche dans le champ à la google (actuellement on recherche l'expression exact "collège toto"et il serait plus utile de rechercher collège et toto)
  • mettre en gras les informations recherchées
#2

Mis à jour par Jérôme Schneider il y a presque 10 ans

  • Fichier 0001_improved_socialisation_places_search.patch ajouté
  • Patch proposed changé de Non à Oui

Un patch qui devrait corriger ce ticket. Je n'ai finalement pas rajouter la case public / privée vu que ça partie de la recherche.

#3

Mis à jour par Jérôme Schneider il y a presque 10 ans

  • Statut changé de Nouveau à Solution déployée
  • Assigné à mis à Jérôme Schneider
#4

Mis à jour par Frédéric Péters il y a presque 10 ans

Je ne sais pas ce qui est entendu par "Solution deployed" dans le cadre de Calebasse, mais je ne vois pas le commit dans le dépôt (http://repos.entrouvert.org/calebasse.git/). Je regardais parce qu'il manque au patch attaché à ce ticket la migration qui ajoute le display_name et l'alimente comme il l'est dans le save().

#5

Mis à jour par Jérôme Schneider il y a presque 10 ans

  • Fichier 0001_improved_socialisation_places_search.patch supprimé
#6

Mis à jour par Jérôme Schneider il y a presque 10 ans

git diff n'ajoute pas les nouveaux fichiers. J'ai finalement utilisé un git format-patch.

#7

Mis à jour par Serghei Mihai il y a presque 10 ans

Je mettrais toutes les chaînes concatenées dans display_name en unicode: par exemple u" (Public)"

#8

Mis à jour par Frédéric Péters il y a presque 10 ans 

Running migrations for ressources:
 - Migrating forwards to 0009_auto__add_field_school_display_name.
 > ressources:0009_auto__add_field_school_display_name
FailedDryRun:  ! Error found during dry run of '0009_auto__add_field_school_display_name'! Aborting.
[...]

Dans ma version de django/south, on ne peut pas accéder ainsi à l'ORM depuis une migration. On peut y accéder via self.orm.School, mais ça donne la situation d'avant la migration. Pour ce qu'il faut faire ici, il faut passer par une migration supplémentaire, dont le rôle serait de passer sur tous les éléments pour en faire un save().

http://south.readthedocs.org/en/latest/tutorial/part3.html#data-migrations

#9

Mis à jour par Frédéric Péters il y a presque 10 ans

Allo? Ce patch qui d'après moi ne marche pas se trouve dans le dépôt, et il y a déjà par dessus un autre patch, qui laisse penser que le commentaire de Serghei a été lu, mais pas la suite.

#10

Mis à jour par Jérôme Schneider il y a presque 10 ans

On Fri, Jul 11, 2014 at 01:47:36PM +0200, wrote:

Allo? Ce patch qui d'après moi ne marche pas se trouve dans le dépôt, et il y a déjà par dessus un autre patch, qui laisse penser que le commentaire de Serghei a été lu, mais pas la suite.

Désolé j'avais bien lu ton commentaire mais je suis repassé sur Vincennes.
La migration fonctionne pour Serghei et pour moi.
C'est un problème de version de South.
J'ai tout de même préparer un patch pour ça et pour forcer la version de
south en >= 0.8.4 (je n'ai juste pas eu le temps de finaliser).

#11

Mis à jour par Frédéric Péters il y a presque 10 ans

Pour info ma version de south est également la 0.8.4.

Et si ça fonctionne, très bien, mais je ne vois alors pas de nécessité de patch (sauf modifier une ligne dans le requirements.txt, mais ça m'étonnerait que ça soit ça qui prenne du temps à être finalisé).

#12

Mis à jour par Benjamin Dauvergne il y a presque 10 ans

Ce qu'il manque ici c'est l'option no_dry_run = True dans la migration. On peut aussi faire:

    if not db.dry_rune:
        ....

autour du code qui modifie la base. Voir http://south.readthedocs.org/en/latest/commands.html pour --db-dry-run.

#13

Mis à jour par Frédéric Péters il y a presque 10 ans

Il y a eu vendredi ce commit de Jérôme : ressources: test db.dry_run on migration 0009, http://repos.entrouvert.org/calebasse.git/commit/?id=26fb289560a20c268419d448031fc9a82a22675f

Ce ticket peut-il ainsi être fermé ?

#14

Mis à jour par Jérôme Schneider il y a presque 10 ans

  • Statut changé de En cours à Résolu (à déployer)
  • % réalisé changé de 80 à 90
#15

Mis à jour par Mikaël Ates (de retour le 29 avril) il y a plus de 9 ans

  • Statut changé de Résolu (à déployer) à En cours

Lors de l'ajout d'une périodie de socialisation, le retour se fait sur l'onglet général du dossier et pas sur l'onglet socialisation.

#16

Mis à jour par Jérôme Schneider il y a plus de 9 ans

  • Statut changé de En cours à Information nécessaire

Je viens de tester sur Firefox et Chromium. J'ai ajouté des périodes de socialisations et je retourne bien dans l'onglet Socialisation et non Général. J'ai également testé avec une erreur dans le formulaire sans plus de succès.

#17

Mis à jour par Mikaël Ates (de retour le 29 avril) il y a plus de 9 ans

  • Statut changé de Information nécessaire à Fermé

Ok

Formats disponibles : Atom PDF