Hi all,

1) Although it isn't a formal sprint as most people see it, as I have been away from my CVS access for sometime, I will be making an effort for the next week to review and commit patches. This could be a good opportunity for you to set aside some time to work on any issues you particularly care about and get quick reviews. Do feel free to contact me directly if you want to discuss a particular issue or patch.

2) This is notice that I intend to release another snapshot release for D6 (6.x-1.34) in a couple of weeks time. I'm keen that these releases are as bug free as possible, so towards the end of this week I will publish on this issue a list of issues that will the release will wait for - issues will only go on this list if they already have a patch (can need work), or are major bugs. Once this list is completed, I'll allow a short 'freeze' period to test for people to test upgrading their sites, and if all seems to go smoothly, will release. I know this sounds like a hassle, but I know that the stability of the stable releases is very important, so am keen that we actively try to prevent releasing with new bugs.

Comments

juliangb’s picture

juliangb commented

Issues that will be included in 6.x-1.34:
(The release will not be made until these are fixed)
#567558: Extend auto-billing to other types of Storm node (feature)
#870428: Reinstate commented out test assertion (bug)
#847166: Invalid string format in query for PostgreSQL (bug)
#857154: Multiple copies of same invoice are display in Invoice View (bug)
#886132: Tests for Storm Task (feature)

(I'll edit these lists during this week for issues that seem to be making progress.)

juliangb’s picture

juliangb commented
Title: Sprint / Release of 6.x-1.34 » Release of 6.x-1.34
Assigned: Unassigned » juliangb

It seems that most of these have been done, so I will now confirm the final list of "release blockers" above.

Once all of those issues have been committed (two left), we will freeze committing for about a week apart from:
- Adding tests where no changes to the code are required to make them pass.
- Bug fixes for bugs that have been introduced since the last stable release.

Whilst this procedure adds hassle, I hope it will help to ensure that Storm stable releases really are stable.

tchurch’s picture

tchurch commented

Can we also check that the bug created in 1.33 for the date issue (with tasks and invoices) is also in 1.34 correctly (see issue #787822: Implement support for Date popup module on Ticket and Invoice Date Fields.
I think it's already corrected in the current -dev version but it could do with some independent confirmation that the dates work again.

juliangb’s picture

juliangb commented
Status: Active » Needs review

All the issues from #1 are complete, and I have checked the bug from #3 (tchurch - perhaps you could confirm too?).

Therefore, I'll now say that we'll freeze committing apart from the two things listed in #2.

The -dev release will be up to date from midnight GMT tonight - from then, I'd really appreciate people confirming that they are testing the latest version. If new bugs are found, please open new issues for them, but cross reference them here if they have been introduced since the last release.

Thanks.

tchurch’s picture

tchurch commented

In relation to #3, I've also tested this and the bug has gone. We're now left with the original bug reported which is easier to cope with if not fixed in this release.

tchurch’s picture

tchurch commented

I can try testing it over the next few days.

tchurch’s picture

tchurch commented

testing update: under #567558: Extend auto-billing to other types of Storm node some problems found with adding projects/tasks/tickets to invoices.
Will supply a small patch again that issue later today for checking.

juliangb’s picture

juliangb commented

I've just committed from #567558: Extend auto-billing to other types of Storm node.

Its only a minor change, but will keep this for another couple of days til releasing for further testing.

A reminder to all:
The current -dev release is almost exactly what will be released as an official snapshot in a few days time - please do try testing it now on a copy of your site.

tchurch’s picture

tchurch commented

I've been doing some testing on the latest -dev release (2010-Aug-31) and it seems to be going OK.
All the invoicing seems to work OK and I've started playing with the permission testing too and that also seems OK.

tchurch’s picture

tchurch commented

Found an issue relating to adding expenses to invoices (not changed with #567558: Extend auto-billing to other types of Storm node but the issue is covered by #695976: Add billable/billed flags to expenses so it's not new.
We could look at fixing #695976: Add billable/billed flags to expenses now or in the next version (1.35).

juliangb’s picture

juliangb commented

If it isn't new, I think we should release and complete this in due course.

I was thinking that the release could ship anytime now. Will try to do it when I get a minute (a bit tied up atm) - or tchurch, you should be able to too if you have a minute sooner than I do.

juliangb’s picture

juliangb commented
Status: Needs review » Fixed

Released.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.