balanced / www.balancedpayments.com

The website for Balanced.
https://www.balancedpayments.com
Other
29 stars 51 forks source link

Push to card #197

Closed kyungmin closed 10 years ago

kyungmin commented 10 years ago

PR for #195

Todo:

kyungmin commented 10 years ago

@mjallday @cohitre any suggestions on how to fix any of the items above?

fangpenlin commented 10 years ago

I like the over all design, but for the push to card circle, somehow it gives a plain feeling to me (I think my brain expects some more details, or something more interesting). Anyway, just personal feeling.

mjallday commented 10 years ago

@kyungmin it would be cool if the open issues on the preview page would have some details. that way if some kind soul wanted to contribute to our progress they could help us by implementing push to card on their favourite client library.

matin commented 10 years ago

@kyungmin In the tokenization response: card_type should be type. That's the only change from when the example in the Crowdhoster campaign was defined.

kyungmin commented 10 years ago

@mjallday it's good to go

matin commented 10 years ago

What about sorting the repos based number of open issues?

mjallday commented 10 years ago

@kyungmin good to go as in ready to deploy? or the issues are filled in? because it doesn't look like the issues have any more information than they did before.

matin commented 10 years ago

@mjallday ready for deploy

On Tuesday, May 6, 2014, Marshall Jones notifications@github.com wrote:

@kyungmin https://github.com/kyungmin good to go as in ready to deploy? or the issues are filled in? because it doesn't look like the issues have any more information than they did before.

— Reply to this email directly or view it on GitHubhttps://github.com/balanced/www.balancedpayments.com/pull/197#issuecomment-42306170 .