search

OCA / vertical-travel

GNU Affero General Public License v3.0
33 stars 74 forks source link

[UPD] Add the job name after the passenger in the report #25

Closed ehdem closed 9 years ago

ehdem commented 9 years ago

Display the job name in the report.

coveralls commented 9 years ago

Coverage Status

Coverage decreased (-0.44%) to 65.34% when pulling 63aff4683cb1cea4c884a162f0dc17b727b6338a on savoirfairelinux:7.0_travel_update_report into 2ee42bc9eed7391c4b954885675311947dceb5f6 on OCA:7.0.

coveralls commented 9 years ago

Coverage Status

Coverage decreased (-0.44%) to 65.34% when pulling 63aff4683cb1cea4c884a162f0dc17b727b6338a on savoirfairelinux:7.0_travel_update_report into 2ee42bc9eed7391c4b954885675311947dceb5f6 on OCA:7.0.

pedrobaeza commented 9 years ago

:+1:

ehdem commented 9 years ago

@pedrobaeza : thanks for your comments

coveralls commented 9 years ago

Coverage Status

Coverage decreased (-0.65%) to 65.13% when pulling 0de53103efc2bab28e253555841af4d9a4ae07a2 on savoirfairelinux:7.0_travel_update_report into 2ee42bc9eed7391c4b954885675311947dceb5f6 on OCA:7.0.

coveralls commented 9 years ago

Coverage Status

Coverage decreased (-0.65%) to 65.13% when pulling 0de53103efc2bab28e253555841af4d9a4ae07a2 on savoirfairelinux:7.0_travel_update_report into 2ee42bc9eed7391c4b954885675311947dceb5f6 on OCA:7.0.

coveralls commented 9 years ago

Coverage Status

Coverage decreased (-0.65%) to 65.13% when pulling 0de53103efc2bab28e253555841af4d9a4ae07a2 on savoirfairelinux:7.0_travel_update_report into 2ee42bc9eed7391c4b954885675311947dceb5f6 on OCA:7.0.

pedrobaeza commented 9 years ago

You're welcome!

bwrsandman commented 9 years ago

Coverage decreased (-0.65%)

Could you add a unittest for _get_passenger for both the correct case and exception cases?

ehdem commented 9 years ago

@bwrsandman: Ok

yvaucher commented 9 years ago

Added need fixing waiting on the test.

bwrsandman commented 9 years ago

:+1: Good work