Résolution de l'issue #3896 #3999

azze-r · 2016-11-21T21:55:03Z

Q	R
correction de bug /	Oui
nouvelle fonctionnalité /evol	Non
Ticket (issue) concerné	(#3896 )

Refactorisation du code

= HelpWriting devient EditorialHelp = tablelabel devient table_label = EditorialHelp déplace de /zds-site/zds/utils/models.py vers /zds-site/zds/tutorialv2/models/models_versioned.py ( #3896 )

- ContentsWithHelps devient HelpContent - Renvoi vers la ZEP en commentaires supprimé - Refactorisation des strings en dur dans le if/elif en important le module zds.tutorialv2.models afin de pouvoir utiliser les valeurs de TYPE_CHOICES à la place des str 'TUTORIAL' et 'ARTICLE' codées en durs - Pluralisation du nom de la class HelpContent via la classe META, qui devient donc : help-contents au pluriel

artragis · 2016-11-22T11:53:08Z

zds/member/migrations/0007_auto_20161119_2307.py

+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('member', '0006_auto_20161119_1650'),


il faut que tu renommes ton fichier 0008_auto_20161119_2307.py et que tu transformes cette ligne en ('member', '0007_auto_20161119_1836'), et le build fonctionnera.

= HelpWriting devient EditorialHelp = tablelabel devient table_label = EditorialHelp déplace de /zds-site/zds/utils/models.py vers /zds-site/zds/tutorialv2/models/models_versioned.py ( #3896 )

- ContentsWithHelps devient HelpContent - Renvoi vers la ZEP en commentaires supprimé - Refactorisation des strings en dur dans le if/elif en important le module zds.tutorialv2.models afin de pouvoir utiliser les valeurs de TYPE_CHOICES à la place des str 'TUTORIAL' et 'ARTICLE' codées en durs - Pluralisation du nom de la class HelpContent via la classe META, qui devient donc : help-contents au pluriel

pierre-24 · 2016-11-23T19:24:19Z

Je pense que Travis est pas content à cause de cette ligne, ou je pense qu'on devrait plutôt avoir table_label à la place (note au passage que je suis pas hyper-fan du nom, j'aurais préféré description ou un truc du genre, vu ce que ça fait)

pierre-24 · 2016-11-23T19:25:32Z

... Et que j'y pense, si le modèle a été déplacé dans tutorialv2, alors la Factory doit y aller de même :)

vhf · 2016-11-23T19:41:04Z

Je suis aussi en faveur de renommer tablelabel/table_label en description tant qu'on y est.

Et pense à supprimer les 3 zip de cette PR. ;)

azze-r · 2016-11-24T22:35:44Z

Merci pour les précisions, j'ai pris en compte vos com's pour le refactor, j'espere juste que j'aurai pas trop de probleme avec les fichiers de migrations :x

azze-r · 2016-11-24T22:42:12Z

Pierre, peux-tu justifier le fait de deplacer la factory? Je ne vois pas très bien pourquoi, as-tu des arguments pour me convaincre? 👍

pierre-24 · 2016-11-25T07:15:13Z

Eh bien, si je lis ta PR, je vois que tu as déplacé (et renommé) le modèle HelpWriting dans zds/tutorialv2/models/models_versionned.py (à noter que je l'aurais mieux vu dans zds/tutorialv2/models/models_database.py, parce que l'autre fichier contient les modèles qui manipulent git), alors que tu as laissé la Factory correspondante dans zds/utils/factories.py . Ça ne me semble pas très logique, si je cherche la factory, je m'attend à la trouver avec les autres du module de contenu, donc dans zds/tutorialv2/factories.py.

Et pour régler ton problème de migration, supprime les fichier de migration et refait un makemigrations, ça devrait aller mieux :)

vhf · 2016-11-25T08:52:38Z

scripts/migrations/20160718_02_utf8mb4.sh

@@ -496,7 +496,7 @@ mysql -u zds -p zdsdb << EOF
    # utils_helpwriting
    ALTER TABLE \`utils_helpwriting\` CHANGE title title VARCHAR(20) CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci NOT NULL;
    ALTER TABLE \`utils_helpwriting\` CHANGE slug slug VARCHAR(20) CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci NOT NULL;
-    ALTER TABLE \`utils_helpwriting\` CHANGE tablelabel tablelabel VARCHAR(150) CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci NOT NULL;
+    ALTER TABLE \`utils_helpwriting\` CHANGE description description VARCHAR(150) CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci NOT NULL;


Ça devrait rester tablelabel ici. C'est la procédure à suivre pour passer de la v19 à la v20, donc elle peut pas changer.

j'aurais pas pu deviner :D

Euh.. je suis pas censé supprimé tous les fichiers de migrations?

azze-r · 2016-11-27T18:01:37Z

Qu'entends-tu par "supprime les fichiers de migrations"? Je delete entièrement le repertoire zds-site/zds/tutorialv2/migrations/ ??

artragis · 2016-11-27T18:10:43Z

Salut, non, uniquement le fichier que tu avais créé en fait. Le 27/11/2016 à 19:06, rouani azedine a écrit :

…

***@***.**** commented on this pull request. ------------------------------------------------------------------------ In scripts/migrations/20160718_02_utf8mb4.sh <#3999>: > @@ -496,7 +496,7 @@ mysql -u zds -p zdsdb << EOF # utils_helpwriting ALTER TABLE \`utils_helpwriting\` CHANGE title title VARCHAR(20) CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci NOT NULL; ALTER TABLE \`utils_helpwriting\` CHANGE slug slug VARCHAR(20) CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci NOT NULL; - ALTER TABLE \`utils_helpwriting\` CHANGE tablelabel tablelabel VARCHAR(150) CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci NOT NULL; + ALTER TABLE \`utils_helpwriting\` CHANGE description description VARCHAR(150) CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci NOT NULL; Euh.. je suis pas censé supprimé tous les fichiers de migrations? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#3999>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABc_xR9Si4xFLncoauxSur_svkSxGHOQks5rCcaRgaJpZM4K4vow>.

azze-r · 2016-11-27T18:13:03Z

euh.. Comment je peux savoir lequel j'ai crée ?

artragis · 2016-11-27T18:13:39Z

C'est le tout dernier en date le plus souvent :) Le 27/11/2016 à 19:13, rouani azedine a écrit :

…

euh.. Comment je peux savoir lequel j'ai crée ? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#3999 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABc_xZZf1xl9fG52Sfft9uoWWkQbppskks5rCcgwgaJpZM4K4vow>.

azze-r · 2016-11-27T18:14:54Z

mdr ;)

azze-r · 2016-11-27T18:15:17Z

Je commence à piger le fonctionnement des fichiers de migrations

azze-r · 2016-11-27T18:21:04Z

Bon, j'ai remis proprement les migrations jusqu'à la numéro 14, maintenant j'essaye de générer derniere migration :
python manage.py makemigrations

Traceback (most recent call last):
  File "manage.py", line 56, in <module>
    execute_from_command_line(sys.argv)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/__init__.py", line 354, in execute_from_command_line
    utility.execute()
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/__init__.py", line 346, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/base.py", line 394, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/base.py", line 445, in execute
    output = self.handle(*args, **options)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/commands/makemigrations.py", line 63, in handle
    loader = MigrationLoader(None, ignore_no_migrations=True)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/db/migrations/loader.py", line 47, in __init__
    self.build_graph()
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/db/migrations/loader.py", line 321, in build_graph
    _reraise_missing_dependency(migration, parent, e)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/db/migrations/loader.py", line 291, in _reraise_missing_dependency
    raise exc
django.db.migrations.graph.NodeNotFoundError: Migration utils.0010_delete_helpwriting dependencies reference nonexistent parent node (u'tutorialv2', u'0015_auto_20161121_2248')

Pourquoi est-ce qu'il demande la 15ém migrations? ce n'est pas justement cette migration que j'essaye de pondre via makemigrations??

vhf · 2016-11-27T19:44:57Z

La liste des fichiers modifiés, ajoutés, supprimés par PR est ici : https://github.com/zestedesavoir/zds-site/pull/3999/files

Supprime les fichiers zip et les migrations présents dans cette liste. Puis génère les migrations, ajoute le tout, et commit. :)

azze-r · 2016-12-20T21:43:54Z

Décidemment je ne vois pas comment régler le pb, quand je fais
python manage.py makemigrations

j'obtiens :

Traceback (most recent call last):
  File "manage.py", line 56, in <module>
    execute_from_command_line(sys.argv)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/__init__.py", line 354, in execute_from_command_line
    utility.execute()
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/__init__.py", line 346, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/base.py", line 394, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/base.py", line 445, in execute
    output = self.handle(*args, **options)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/commands/makemigrations.py", line 63, in handle
    loader = MigrationLoader(None, ignore_no_migrations=True)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/db/migrations/loader.py", line 47, in __init__
    self.build_graph()
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/db/migrations/loader.py", line 321, in build_graph
    _reraise_missing_dependency(migration, parent, e)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/db/migrations/loader.py", line 291, in _reraise_missing_dependency
    raise exc
django.db.migrations.graph.NodeNotFoundError: Migration utils.0010_delete_helpwriting dependencies reference nonexistent parent node (u'tutorialv2', u'0015_auto_20161121_2248')
(zdsenv) [boblinux@boblinux-pc zds-site]$ python manage.py migrations
Unknown command: 'migrations'
Type 'manage.py help' for usage.
(zdsenv) [boblinux@boblinux-pc zds-site]$ python manage.py migration
Unknown command: 'migration'
Type 'manage.py help' for usage.
(zdsenv) [boblinux@boblinux-pc zds-site]$ python manage.py migrate
Traceback (most recent call last):
  File "manage.py", line 56, in <module>
    execute_from_command_line(sys.argv)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/__init__.py", line 354, in execute_from_command_line
    utility.execute()
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/__init__.py", line 346, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/base.py", line 394, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/base.py", line 445, in execute
    output = self.handle(*args, **options)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/core/management/commands/migrate.py", line 93, in handle
    executor = MigrationExecutor(connection, self.migration_progress_callback)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/db/migrations/executor.py", line 19, in __init__
    self.loader = MigrationLoader(self.connection)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/db/migrations/loader.py", line 47, in __init__
    self.build_graph()
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/db/migrations/loader.py", line 321, in build_graph
    _reraise_missing_dependency(migration, parent, e)
  File "/home/boblinux/github_codes/zdsenv/lib/python2.7/site-packages/django/db/migrations/loader.py", line 291, in _reraise_missing_dependency
    raise exc
django.db.migrations.graph.NodeNotFoundError: Migration utils.0010_delete_helpwriting dependencies reference nonexistent parent node (u'tutorialv2', u'0015_auto_20161121_2248')

Et je n'ai pas de nouvelles migrations vu que mes migrations s'arrêtent à la migration 14 ...

Anto59290 · 2017-01-20T07:49:37Z

@Boblinux : toujours motivé pour finir cette PR ? Si oui on doit probablement pouvoir t'aider à finaliser ça.

azze-r · 2017-01-22T22:16:16Z

Bien sûr, je suis chaud mais je me sens un peu "délaissé" pour le coup, c'est sûr que ça démotive de n'avoir aucun retour pendant 1 mois lors de notre première PR )=' mais peut-être que là je vais reprendre espoir ❌

Anto59290 · 2017-01-23T08:08:53Z

Ton dernier problème est le suivant : ton fichier de migration zds/utils/migrations/0010_delete_helpwriting.py référence un autre fichier le : ('tutorialv2', '0015_auto_20161121_2248'). Mais ce dernier n'existe pas. Une solution est de supprimer ton fichier de migration (le numéro 10 dans le répertoire Utils/migrations) et de recréer les migrations. Ca devrait régler le problème. N'oublie pas qu'il existe un dossier de migration par module (et donc par répertoire de /zds)

Autre soucis noté par @vhf, tu envoi dans cette PR des nouveaux fichiers .zip d'archive de tutos. Normalement tu n'as pas de raison de les modifier (ou alors il faut expliquer pourquoi). Il faut donc que tu supprime ces fichiers Zip de ta PR.

N'hésites pas si tu as des questions ;)

gllmc · 2017-09-02T19:40:46Z

@Boblinux : es-tu toujours motivé par cette PR ? As-tu besoin d'aide ?

azze-r · 2017-09-07T21:00:50Z

Je ne suis plus sur le coup désolé, actuellement je suis en master 2 info du coup le temps me manque énormément, merci à tous :) 2017-09-02 21:40 GMT+02:00 Guillaume <[email protected]>:

…

@Boblinux <https://github.com/boblinux> : es-tu toujours motivé par cette PR ? As-tu besoin d'aide ? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3999 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AJ4BBw_YTshI8XpnJWW0DfUXLdc8bRHqks5sea8_gaJpZM4K4vow> .

gllmc · 2017-09-08T17:23:39Z

Ok, pas de problème, je vais fermer. Si quelqu'un souhaite reprendre, n'hésitez pas. :)

azze-r added 4 commits November 21, 2016 21:51

Refactore le nom de la classe HelpWriting par EditorialHelp (#3896)

847bcd0

- HelpWriting

d4a04c7

= HelpWriting devient EditorialHelp = tablelabel devient table_label = EditorialHelp déplace de /zds-site/zds/utils/models.py vers /zds-site/zds/tutorialv2/models/models_versioned.py ( #3896 )

makeMigrations effectué car on a modifié le modèle (#3896)

2afc700

artragis reviewed Nov 22, 2016

View reviewed changes

azze-r added 6 commits November 22, 2016 21:06

migrations change 0006_auto_20161119_1650 to 0007_auto_20161119_1836

d01f0bb

Refactore le nom de la classe HelpWriting par EditorialHelp (#3896)

59d7fa5

- HelpWriting

38c1678

= HelpWriting devient EditorialHelp = tablelabel devient table_label = EditorialHelp déplace de /zds-site/zds/utils/models.py vers /zds-site/zds/tutorialv2/models/models_versioned.py ( #3896 )

makeMigrations effectué car on a modifié le modèle (#3896)

e684eeb

migrations change 0006_auto_20161119_1650 to 0007_auto_20161119_1836

4f999d4

azze-r added 2 commits November 24, 2016 23:23

Clean refactor table_label to description (#3896)

e06afae

delete migration file

74d3fda

clean git code generated to merge

8e1fc87

vhf reviewed Nov 25, 2016

View reviewed changes

vhf added C-Back Concerne le back-end Django S-Refactorisation Améliore le code existant sans forcément ajouter de nouvelle fonctionnalité labels Dec 24, 2016

gllmc closed this Sep 8, 2017

philippemilink mentioned this pull request May 18, 2024

Que doit être affiché dans la liste des demandes d'aides pour les contenus ? #6616

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Résolution de l'issue #3896 #3999

Résolution de l'issue #3896 #3999

azze-r commented Nov 21, 2016

artragis Nov 22, 2016

pierre-24 commented Nov 23, 2016

pierre-24 commented Nov 23, 2016

vhf commented Nov 23, 2016

azze-r commented Nov 24, 2016

azze-r commented Nov 24, 2016

pierre-24 commented Nov 25, 2016

vhf Nov 25, 2016

azze-r Nov 27, 2016

azze-r Nov 27, 2016

azze-r commented Nov 27, 2016

artragis commented Nov 27, 2016 via email

azze-r commented Nov 27, 2016

artragis commented Nov 27, 2016 via email

azze-r commented Nov 27, 2016

azze-r commented Nov 27, 2016

azze-r commented Nov 27, 2016

vhf commented Nov 27, 2016

azze-r commented Dec 20, 2016

Anto59290 commented Jan 20, 2017

azze-r commented Jan 22, 2017

Anto59290 commented Jan 23, 2017

gllmc commented Sep 2, 2017

azze-r commented Sep 7, 2017 via email

gllmc commented Sep 8, 2017

Résolution de l'issue #3896 #3999

Résolution de l'issue #3896 #3999

Conversation

azze-r commented Nov 21, 2016

artragis Nov 22, 2016

Choose a reason for hiding this comment

pierre-24 commented Nov 23, 2016

pierre-24 commented Nov 23, 2016

vhf commented Nov 23, 2016

azze-r commented Nov 24, 2016

azze-r commented Nov 24, 2016

pierre-24 commented Nov 25, 2016

vhf Nov 25, 2016

Choose a reason for hiding this comment

azze-r Nov 27, 2016

Choose a reason for hiding this comment

azze-r Nov 27, 2016

Choose a reason for hiding this comment

azze-r commented Nov 27, 2016

artragis commented Nov 27, 2016 via email

azze-r commented Nov 27, 2016

artragis commented Nov 27, 2016 via email

azze-r commented Nov 27, 2016

azze-r commented Nov 27, 2016

azze-r commented Nov 27, 2016

vhf commented Nov 27, 2016

azze-r commented Dec 20, 2016

Anto59290 commented Jan 20, 2017

azze-r commented Jan 22, 2017

Anto59290 commented Jan 23, 2017

gllmc commented Sep 2, 2017

azze-r commented Sep 7, 2017 via email

gllmc commented Sep 8, 2017