DGNum/gestioCOF

Permet de transférer/réinitialiser les reventes #476

Merged

lstephan merged 0 commits from Aufinal/fix_revente into master

2017-02-22 14:08:14 +01:00

lstephan commented

2017-02-16 04:56:08 +01:00

(Migrated from git.eleves.ens.fr)

fixes #122

fixes #122

lstephan commented

2017-02-16 05:29:15 +01:00

(Migrated from git.eleves.ens.fr)

Added 1 commit:

d524d928 - better annulation check

Added 1 commit: * d524d928 - better annulation check

lstephan commented

2017-02-16 12:22:29 +01:00

(Migrated from git.eleves.ens.fr)

Added 1 commit:

8953d3de - remove immediate mail

Added 1 commit: * 8953d3de - remove immediate mail

lstephan commented

2017-02-16 12:56:01 +01:00

(Migrated from git.eleves.ens.fr)

Added 1 commit:

d96f4ead - Revert "remove immediate mail"

Added 1 commit: * d96f4ead - Revert "remove immediate mail"

mpepin commented

2017-02-21 16:17:28 +01:00

(Migrated from git.eleves.ens.fr)

La PEP8 n'aime pas trop les \ quand on peut s'en passer cf https://www.python.org/dev/peps/pep-0008/#maximum-line-length. C'est pas critique, je te laisse aviser si tu changes ou laisse mais j'aurais plutôt écrit quelque chose comme

self.fields['attributions'].queryset = (
    participant.attribution_set.filter(
        revente__isnull=False,
        revente__soldTo__isnull=False)
    .exclude(revente__soldTo=participant)
)

La PEP8 n'aime pas trop les `\` quand on peut s'en passer cf https://www.python.org/dev/peps/pep-0008/#maximum-line-length. C'est pas critique, je te laisse aviser si tu changes ou laisse mais j'aurais plutôt écrit quelque chose comme self.fields['attributions'].queryset = ( participant.attribution_set.filter( revente__isnull=False, revente__soldTo__isnull=False) .exclude(revente__soldTo=participant) )

mpepin commented

2017-02-21 16:20:24 +01:00

(Migrated from git.eleves.ens.fr)

On n'en profiterait pas pour déplacer ce template dans bda/templates/bda/ ?

On n'en profiterait pas pour déplacer ce template dans `bda/templates/bda/` ?

mpepin commented

2017-02-21 16:22:23 +01:00

(Migrated from git.eleves.ens.fr)

Modulo les deux points précédents c'est bon pour moi

Modulo les deux points précédents c'est bon pour moi

lstephan commented

2017-02-21 23:49:47 +01:00

(Migrated from git.eleves.ens.fr)

Added 1 commit:

42a93027 - fix pep8 + move template

Added 1 commit: * 42a93027 - fix pep8 + move template

mpepin commented

2017-02-22 14:08:13 +01:00

(Migrated from git.eleves.ens.fr)

Status changed to merged

Status changed to merged

mpepin commented

2017-02-22 14:08:14 +01:00

(Migrated from git.eleves.ens.fr)

mentioned in commit a0a7be43e1

mentioned in commit a0a7be43e10b8ed71ca86911e67772f7ce29cac5

Sign in to join this conversation.

No reviewers

No labels

devtype -- backend

devtype -- docs

devtype -- frontend

devtype -- user interface

difficulty -- easy

difficulty -- hard

difficulty -- normal

Good first issue

priority -- high

priority -- low

priority -- medium

priority -- staff-wanted

status -- development

status -- discussion

status -- need review

status -- production

status -- ready to merge

type -- hygiene

type -- improvement

type -- new feature

No milestone

No project

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/gestioCOF#476

No description provided.