Skip to content

Dave Clark
clarkdave

Organizations

@lrd @loyaltylion
@clarkdave
clarkdave commented on pull request loyaltylion/loyaltylion-magento#4
@clarkdave

:star:

clarkdave commented on pull request loyaltylion/loyaltylion-magento#3
@clarkdave

:star2:

clarkdave commented on pull request loyaltylion/loyaltylion-magento#3
@clarkdave

Before merging (if you haven't check already) does it pass our standard checklist in Magento 1.7, 1.8 and 1.9 (via the VMs)?

clarkdave commented on pull request loyaltylion/loyaltylion-magento#3
@clarkdave

where does this Mage::log thing end up?

clarkdave commented on pull request loyaltylion/loyaltylion-magento#3
@clarkdave

I think this should be camelCase? Not that we have a PHP style guide, but everything else is?

clarkdave commented on pull request loyaltylion/loyaltylion-magento#3
@clarkdave

Is this important? Or just a tidyup thing?

clarkdave commented on pull request loyaltylion/loyaltylion-magento#2
@clarkdave

:star2:

Something went wrong with that request. Please try again.