On 1/18/06, Adrian Holovaty <[EMAIL PROTECTED]> wrote:
Ok; fair comment. I was working on the assumption that there had been a reasonable amount of activity on the Django-developer list that day, so all the main players were around, and if there was a comment to be made, someone would have at least said 'not sure.. lets look into it'. Coupled with the fact that the patch seemed reasonably trivial to me - of course, I missed related tables....
Would the right approach been to wait longer, or to ask somewhere else?
Ok; I overlooked that bit. I'll see what I can do.
Do you want me to roll out the bulk delete implementation in the interim?
Russ Magee %-)
On 1/18/06, hugo <[EMAIL PROTECTED]> wrote:
> >Ok; in the absence of any objections, I've just committed the change,
> >and closed off the ticket. Bulk delete is now available in
> >magic-removal.
>
> Hum. That's a _tad_ bit fast committing for me - just because nobody
> commented on it doesn't say there aren't any comments on it in the
> queue :-)
Yeah, that's a tad bit fast for me, as well. A lack of comments
doesn't give consent.
Ok; fair comment. I was working on the assumption that there had been a reasonable amount of activity on the Django-developer list that day, so all the main players were around, and if there was a comment to be made, someone would have at least said 'not sure.. lets look into it'. Coupled with the fact that the patch seemed reasonably trivial to me - of course, I missed related tables....
Would the right approach been to wait longer, or to ask somewhere else?
The patch doesn't appear to handle this, which means we have
inconsistent behavior between bulk delete and normal deletion.
Ok; I overlooked that bit. I'll see what I can do.
Do you want me to roll out the bulk delete implementation in the interim?
Russ Magee %-)
