savoirfairelinux / sous-chef

Sous-Chef is a web application to help organizations to plan and deliver meals, and to manage clients files.
GNU Affero General Public License v3.0
66 stars 45 forks source link

Fixed #717: Enhance client scheduled status change #730

Closed lingxiaoyang closed 7 years ago

lingxiaoyang commented 7 years ago

Fixes #717 by lingxiaoyang

Changes proposed in this pull request:

Status

How to verify this change

See above.

Deployment notes and migration

New translatable strings

Additional notes

none

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.3%) to 94.915% when pulling 657fae718b6c122b03332deecffb7d17bf9a0a52 on lingxiaoyang:fix_717 into 1f36579921e330e8706346d642370208c285c6d1 on savoirfairelinux:dev.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.1%) to 95.341% when pulling 0ed569c94b75f72f80dc22ee8e1cce62b69d9583 on lingxiaoyang:fix_717 into bbfb37a81e5cd2e80cb08df0ace29b3c35290faa on savoirfairelinux:dev.

lingxiaoyang commented 7 years ago

Removed "WIP" flag. Unit tests and translatable strings are added.

erozqba commented 7 years ago

@lingxiaoyang could you resolve the conflicts, please? Thanks in advance.

lingxiaoyang commented 7 years ago

@erozqba Thank you! I didn't notice that..

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.1%) to 95.39% when pulling 22cb0d9ae47fa6d1524025623e17dd0f90b1c52e on lingxiaoyang:fix_717 into 395ac8252a1de669d46b518f459ba1093a9d47cb on savoirfairelinux:dev.

lingxiaoyang commented 7 years ago

@erozqba OK!

lamontfr commented 7 years ago

@lingxiaoyang I tested all the cases and it works perfectly. Thanks Ling for this excellent work ! You can now merge this PR.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.1%) to 95.415% when pulling acda830cd72585d13165f91f053ec984ac5ba338 on lingxiaoyang:fix_717 into 7e4fafc4e2576e843192eddaa36317ca987dd3e0 on savoirfairelinux:dev.

lingxiaoyang commented 7 years ago

@lamontfr Thank you! I rebased in acda830. I'm merging it now.