Skip to content

Conversation

@SimonSarazin
Copy link
Member

@Karmak23 , je suis pas sûr de moi sur cette pull request finalement. Je vais peut-être supprimer ma branche et repartir de master

SimonSarazin and others added 22 commits April 17, 2015 11:24
Also deleted "groups" as we'll not use Group app
Update usage API so that's faster to charge API
…{begin,end}_date.

WARNING: south tried to add the `created_on` column, but I already add it in my database, thus I removed the field from the migration.
Fixes last name of myself
Also deleted "groups" as we'll not use Group app
…into develop

Conflicts:
	projects/api.py
	projects/models.py
@Karmak23
Copy link
Contributor

LOL. Clairement, github devrait nous montrer 1 seul patch dans la PR. Là, « some commits » (vu le nombre), ça ne me dit rien qui vaille au merge ;-) Effectivement un petit reboot depuis master (ou develop) devrait mettre les choses à plat.

checkout develop dans un autre répertoire, et utilises meld pour voir les différences avec ton repo actuel. Tu ne devrais voir que les choses qui ont changé, et il en résultera un patch propre.

PS: si tu as besoin d'aide on peut se le faire à deux. Je serai dispo en fin d'aprem.

@SimonSarazin
Copy link
Member Author

Ok, je repars d'un truc clean. Pas d'urgence de toute manière, on a rien
besoin de ce que j'allais envoyer, c'est juste du nettoyage.

On Mon, Apr 20, 2015 at 1:37 PM Olivier Cortès [email protected]
wrote:

LOL. Clairement, github devrait nous montrer 1 seul patch dans la PR. Là,
« some commits » (vu le nombre), ça ne me dit rien qui vaille au merge ;-)
Effectivement un petit reboot depuis master (ou develop) devrait mettre les
choses à plat.

checkout develop dans un autre répertoire, et utilises meld pour voir les
différences avec ton repo actuel. Tu ne devrais voir que les choses qui ont
changé, et il en résultera un patch propre.


Reply to this email directly or view it on GitHub
#37 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants