Skip to content
This repository has been archived by the owner on May 7, 2021. It is now read-only.

Add Blogger Gadget #79

Merged
merged 7 commits into from
Feb 6, 2015
Merged

Add Blogger Gadget #79

merged 7 commits into from
Feb 6, 2015

Conversation

techtonik
Copy link
Contributor

You can see how it works at http://techtonik.rainforce.org/2014/03/python-c-apiabi-compatibility-report.html on the left under navigation.

@techtonik techtonik mentioned this pull request Jan 11, 2015
@rummik
Copy link
Contributor

rummik commented Jan 13, 2015

I don't use Blogger. Everything looks okay to me though. If nobody has any objections, I'll merge it and publish.

@whit537

@@ -0,0 +1,12 @@
gratipay.xml is a an example of Gadget for Blogger. To add it, go to
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo: a an example of Gadget -> an example of a Gadget

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Long time no see. Fixed. =)

@techtonik
Copy link
Contributor Author

Ready for review.

@chadwhitacre
Copy link
Contributor

Ping @rummik. Can you review/merge this pls? :)

@@ -16,8 +16,7 @@ $ cd grtp.co
$ make run
```

Then visit http://localhost:9537/test/

Then visit [http://localhost:9537/test/](http://localhost:9537/test/)
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the one thing I would suggest changing is the link here to use <> notation:

<http://localhost:9537/test/> -> http://localhost:9537/test/

Since it's documentation, and it's only a minor change, this can be done after the merge, so I'm not going to block on it because that would just be silly :P

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cool. Didn't know that syntax exist. =)

This makes this file use default grtp.co "licensing terms", which
is the same stuff, but in legalese.
@techtonik
Copy link
Contributor Author

@rummik, please take a look. =)

@rummik
Copy link
Contributor

rummik commented Feb 6, 2015

Looks good! :3

rummik added a commit that referenced this pull request Feb 6, 2015
@rummik rummik merged commit e74552e into master Feb 6, 2015
@techtonik techtonik deleted the blogger branch February 7, 2015 08:42
@techtonik
Copy link
Contributor Author

Thanks! =)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants