sandstorm-io / blackrock

Cluster management
Apache License 2.0
74 stars 13 forks source link

Add blackrock-payments code for referral program #14

Closed paulproteus closed 8 years ago

paulproteus commented 8 years ago
paulproteus commented 8 years ago

Responded to @jparyani's comment with the extra commit.

paulproteus commented 8 years ago

Dumb (?) question - are we dogmatic about if to indent the {{# if ... }} and also the thing inside it?

Seems like:

So I followed that in a follow-up commit.

paulproteus commented 8 years ago

All fixed, I think.

kentonv commented 8 years ago

HTML indentation is weird because if you indent everything it can get excessively deep. The main rules I'd set are:

Of course, this is all nitpicky. The main thing is it should be legible and not misleading.

kentonv commented 8 years ago

Code looks fine now, will merge after Sandstorm itself is updated.