Skip to content

Conversation

pon
Copy link
Contributor

@pon pon commented Mar 6, 2015

@lob/api-engineering-team

README.md Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wait why is this url in assets.lob.com it shouldnt be in there

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to use ' ' here, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Damn I didn't notice - I just copied it from the docs page

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and that S3 buckets shouldn't have items like this in it

@pon pon force-pushed the add-check-logo branch from e2297a6 to f37e1a9 Compare March 6, 2015 16:39
@pon
Copy link
Contributor Author

pon commented Mar 6, 2015

Updated with new logo link

@leore
Copy link
Contributor

leore commented Mar 6, 2015

lgtm

pon added a commit that referenced this pull request Mar 6, 2015
feat(checks): added docs and example for check logo
@pon pon merged commit ba015c2 into master Mar 6, 2015
@pon pon deleted the add-check-logo branch March 6, 2015 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants