Skip to content
This repository has been archived by the owner on Dec 29, 2022. It is now read-only.

bug fixes for blogger blog id parsing #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

snarfed
Copy link

@snarfed snarfed commented Jun 15, 2015

specifically:

specifically:
* handle new style blogger blog ids prefixed with g. details in snarfed/bridgy#168
* don't crash if blog id can't be parsed. details in bridgy/issues#147
@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@snarfed
Copy link
Author

snarfed commented Jun 15, 2015

done!

@googlebot
Copy link

CLAs look good, thanks!

snarfed added a commit to snarfed/oauth-dropins that referenced this pull request Jun 15, 2015
* move webutil to oauth_dropins.webutil. it's a bit presumptuous to put it at the top level when it's named so broadly.
* switch gdata-python-client from submodule to pip dependency. (i've submitted my two bug fix patches as google/gdata-python-client#27.)
* move appengine_config.py to oauth_dropins, except for vender/virtualenv setup
* update README
@snarfed
Copy link
Author

snarfed commented Jul 19, 2015

friendly ping!

snarfed added a commit to snarfed/oauth-dropins that referenced this pull request Jul 23, 2015
snarfed added a commit to snarfed/bridgy that referenced this pull request Jul 23, 2015
@snarfed
Copy link
Author

snarfed commented Nov 22, 2016

friendly ping? i've been using my fork for the last 18 mos due to this bug. :/ i'd much rather get this merged and use this upstream repo instead!

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.

2 participants