Tim Warberg twarberg

Organizations

@QuickPay
@twarberg
  • @twarberg d635c46
    Do not daemonize carbon-cache
@twarberg
twarberg created repository Monsido/grafana-graphite
twarberg commented on pull request QuickPay/quickpay-php-client#1
@twarberg

@tolvstein - Bør nok også være QuickPay som dir navn.

@twarberg
API url should be configurable
@twarberg
Wrong namespace
twarberg commented on pull request QuickPay/quickpay-php-client#1
@twarberg

Generally looks good except for a few minor comments.

twarberg commented on pull request QuickPay/quickpay-php-client#1
@twarberg

Please don't mix camelCase and snake_case

twarberg commented on pull request QuickPay/quickpay-php-client#1
@twarberg

Would personally prefer namespace QuickPay\API. @ta?

twarberg commented on pull request QuickPay/quickpay-php-client#1
@twarberg

Its considered best practice to omit ?> on pure PHP files (http://www.php-fig.org/psr/psr-2/)

@twarberg
Decode JSON response if not raw!?
@twarberg

Nevermind, the error was elsewhere

@twarberg
Decode JSON response if not raw!?
twarberg opened pull request coherify/ruconomic#1
@twarberg
Added fetching invoice by number
1 commit with 17 additions and 12 deletions
@twarberg
  • @twarberg 2f24b69
    Added fetching invoice by number
@twarberg