search

patacrep / patanet

Web interface for LaTeX songbook generation
GNU Affero General Public License v3.0
10 stars 3 forks source link

New section via js #118

Closed oliverpool closed 9 years ago

oliverpool commented 9 years ago

Fix #111 & Fix #93

L'ajout de section ne nécéssite plus de soumettre le formulaire au serveur : tout est géré en JS. Cependant il n'y a pas d'AJAX : la sauvegarde en BDD est faite lors du clic sur "Enregistrer les changements"

J'en ai profité pour refactoriser complètement la fonction move_or_delete_items.

Luthaf commented 9 years ago

Juste une remarque git : evite de faire trop de merge, utilise plutôt des rebases pour déplacer tes commits au dessus des nouveaux commits qui sont arrivés entre-temps sur la branche d'origine.

Je regarderai le code plus tard !

oliverpool commented 9 years ago

ok (j'ai fait l'erreur de faire un push avant le merge/rebase : du coup faire un rebase peut être génant pour les autres personnes chez qui la branche peut être syncronisée - pas très pertinent dans notre cas, je te l'accorde ;-)

Mais du coup maintenant (que j'ai fait n'importe quoi), il vaut mieux que je refasse un merge de Python3 dans new_section_via_JS, non ? (ça permettra de mieux voir les "Files Changed")

Luthaf commented 9 years ago

Non, par forcément. Si tu fais un rebase suivit d'un push -f, ça nettoie tout et on a bien tous les changements, et pas 10 commits de merge inutiles au milieu.

Et les autres personnes doivent utiliser pull --rebase pour récuperer la nouvelle branche toute popre !

oliverpool commented 9 years ago

Ok, je ne connaissais pas : c'est noté !

oliverpool commented 9 years ago

(ça pertube un peu github, mais à part ça c'est tout bon !)

Luthaf commented 9 years ago

(ça pertube un peu github, mais à part ça c'est tout bon !)

Ça, et les références aux id de commits qui change. Mais sinon j'aime bien !

oliverpool commented 9 years ago

Le rebase est fait, il ne reste plus qu'à merger !

Luthaf commented 9 years ago

Ça m'a l'air bon aussi. Bon boulot en attendant !

oliverpool commented 9 years ago

merci!

Je commence un stage en entreprise dans 8h...donc ma cadence de dev risque de diminuer très fortement dans les prochaines semaines