Opened 8 years ago

Last modified 8 years ago

defect #76 (closed: DONE)

Once a ticket has been partial completed, it won't send notification mail.

Test Complete Size: 5 Test Complete Date: 05/23/2009
Documentation Complete Size: 0 Documentation Complete Date:
Acceptance Complete Size: 3 Acceptance Complete Date: 05/24/2009
Reported by: augustinus Owned by: ja11sop
Milestone: Clean up of Plugin with Patch Component: workflow
Version: Keywords:
Cc: Blocked By:
Blocking: Patch SVN Revision:
Patch Trac Version: not_applicable
In Iterations: 16 19 23

Description

As title described. :)

Change History

Have a look at the list of modified files related to this ticket.

in reply to: ↑ description   Changed 8 years ago by augustinus

Replying to augustinus:

As title described. :)

Especially after being closed/reopen.

  Changed 8 years ago by ja11sop

  • test_complete_size changed from undefined to 5
  • doc_complete_size changed from undefined to 0
  • acceptance_complete_size changed from undefined to 3

Great, thanks for the report. Have added this to the clean-up milestone which will be getting some attention real soon now.

follow-up: ↓ 4   Changed 8 years ago by ja11sop

  • trac_version set to not_applicable

It seems this might only be broken in the later patches. Can you confirm your patch version?

in reply to: ↑ 3   Changed 8 years ago by augustinus

Replying to ja11sop:

It seems this might only be broken in the later patches. Can you confirm your patch version?

I'll check it later and then report details. Thanks.

  Changed 8 years ago by ja11sop

  • test_complete_date set to 05/23/09

This was happening because we internally store the completion dates as actual datetimes. The notification code that sends the emails was expecting strings and so failed when it tried to perform string operations. This has now been fixed.

  Changed 8 years ago by ja11sop

  • acceptance_complete_date set to 05/24/09
  • status changed from sized to closed
  • resolution set to DONE
Note: See TracTickets for help on using tickets.