DGNum/ernestophone.ens.fr
6
0
Fork 0
Code Issues 7 Pull requests 3 Projects Releases Packages Wiki Activity Actions

Hotfix appliqué en prod pour que le site démarre #31

Merged
mpepin merged 3 commits from kerl/hotfix_prod into master 2020-09-12 19:07:13 +02:00
Conversation 10 Commits 3 Files changed 2 +5 -5
mpepin commented 2020-09-12 13:45:57 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

cc @champeno

cc @champeno
mpepin commented 2020-09-12 13:48:54 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Il restait une utilisation du champ .calendrier comme si c'était un booléen.

J'ai mis "F" au pif pour que ça arrête de crasher mais il faut me dire ce que tu veux ici : fanfarons / tout le monde / hall of fame ?

Il restait une utilisation du champ `.calendrier` comme si c'était un booléen. J'ai mis `"F"` au pif pour que ça arrête de crasher mais il faut me dire ce que tu veux ici : fanfarons / tout le monde / hall of fame ?
champeno commented 2020-09-12 13:53:03 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

En fait en toute logique il faudrait ne pas filtrer sur calendrier, on veut afficher tous les évènements pour la vue logged-in. @lgalland un avis ?

En fait en toute logique il faudrait *ne pas* filtrer sur `calendrier`, on veut afficher tous les évènements pour la vue logged-in. @lgalland un avis ?
mpepin commented 2020-09-12 13:53:17 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

On ne peut pas faire "-" + Lower("nom") : Lower(...) n'est pas une chaîne de caractères, tu dois faire Lower("nom").asc() ou Lower("nom").desc() pour choisir l'ordre.

J'ai mis par ordre alphabétique croissant, dis moi si c'est ce que tu veux sinon j'ajoute .desc()

On ne peut pas faire `"-" + Lower("nom")` : `Lower(...)` n'est pas une chaîne de caractères, tu dois faire `Lower("nom").asc()` ou `Lower("nom").desc()` pour choisir l'ordre. J'ai mis par ordre alphabétique croissant, dis moi si c'est ce que tu veux sinon j'ajoute `.desc()`
champeno commented 2020-09-12 13:54:51 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

J'avoue que cette ligne me paraissait douteuse mais elle marchait avec sqlite ^^"

Je pense qu'ordre croissant c'est plus naturel. Pareil @lgalland un avis ?

J'avoue que cette ligne me paraissait douteuse mais elle marchait avec sqlite ^^" Je pense qu'ordre croissant c'est plus naturel. Pareil @lgalland un avis ?
mpepin commented 2020-09-12 13:58:23 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ouais… Sqlite est trop permissif et je suis assez fâché que Django laisse passer ça, l'ORM devrait nous envoyer bouler.

Ouais… Sqlite est trop permissif et je suis assez fâché que Django laisse passer ça, l'ORM devrait nous envoyer bouler.
lgalland commented 2020-09-12 14:31:00 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Je suis d'accord !

Je suis d'accord !
lgalland commented 2020-09-12 14:32:52 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

oui l'ordre croissant est bien :)

oui l'ordre croissant est bien :)
mpepin commented 2020-09-12 19:06:17 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](/klub-dev-ens/Ernesto/-/merge_requests/10/diffs?diff_id=2111&start_sha=8d3805a922c17a0e1d00a7b87aa8ed37b621b9eb#584fbd9c147a2208ce211516796022ad97bed08b_60_58)
mpepin commented 2020-09-12 19:06:17 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

  • c3f9b26a - Show all events to logged-in users

Compare with previous version

added 1 commit <ul><li>c3f9b26a - Show all events to logged-in users</li></ul> [Compare with previous version](/klub-dev-ens/Ernesto/-/merge_requests/10/diffs?diff_id=2111&start_sha=8d3805a922c17a0e1d00a7b87aa8ed37b621b9eb)
mpepin commented 2020-09-12 19:07:13 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

merged

merged
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Doing

   
Good first issue

   
To Do
  
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No labels
Doing
Good first issue
To Do
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: DGNum/ernestophone.ens.fr#31
No description provided.
Delete branch "kerl/hotfix_prod"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?