User talk:Backsword/Sandbox/QI

acceptance reward
After much discussion the consensus did not think this was a good name for this parameter. The name 'early benefits' raised no objections, so that is probably what it should be called. mtew 04:24, 4 April 2009 (UTC)
 * This doesn't touch on that at all. I only copies what was on the template page already. Don't take this as having any relevance one way or other on changing this. Backsword 03:19, 6 April 2009 (UTC)

Categories
'mini-games' need to be properly categorized. mtew 04:24, 4 April 2009 (UTC)
 * I'd rather manually categorise 2 articles that introduce extensive exception code that would make things work in an counterintuiative way. Backsword 03:19, 6 April 2009 (UTC)

pre-Searing
Combining pre-Searing Ascalon quests with post-Searing Ascalon quests is not helpful in my opinion. (I may have misread your coding though.) mtew 04:24, 4 April 2009 (UTC)
 * Can't hrelp you there. It's the same section as that's how it's ingame. Backsword 03:19, 6 April 2009 (UTC)

Change between current
here --JonTheMon 13:16, 6 April 2009 (UTC)

Please document what the effective changes are...
I see you added a mandatory 'section' parameter and dropped the mini-game auto-categorization, but did you do anything else?

The code style changes are a good thing in my opinion.

The section and categorization changes are an agenda item that I disagree with. In most cases the section information is derivative of the type and region values and should not be mandatory. If anything, region and campaign should be mandatory.

mtew 14:47, 9 April 2009 (UTC)
 * Correlation is not causation: the code is only guessing and the parameter must be set if the guessing goes awry. Campaign isn't really optional, it's just that the code is smarter and can provide a default now. I'll chage the document to better relect this. Backsword 08:33, 11 April 2009 (UTC)