Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add Google Image Search feature #35

Merged
merged 1 commit into from
May 28, 2014

Conversation

shao1555
Copy link
Contributor

to use this plugin with Google Image Search API, I've made some modification to this extension.

もう「ググる」と言わないで!:日経ビジネスオンライン

@fukayatsu
Copy link
Owner

Thanks! @shao1555

I want this feature too. (ref: #21 )

However, Google Image Search API has been officially deprecated as of May 26, 2011. And It will become not available soon.

https://developers.google.com/image-search/?csw=1

What do you think about that?

@shao1555
Copy link
Contributor Author

Nihongo de, okay?

利用規約 の Deprecation の節をみるには

  • 3 年間はサービスの提供をするが、機能追加などはしない
  • Google は法律やセキュリティなどの理由で予告なく停止する権利を持つ

とあり、3 年経過してしまったのであまりよくないですね。

README 等に断りをいれてリリースしてしまうのも (お遊び的なアプリなので) 許されるかもしれませんがどうでしょう?

あるいは各自に API キーを用意してもらった上で Google Custom Search API や Bing Search API を叩く実装にしましょうかね。。(設定画面を作るのが少々手間。。)

ミサワ

@fukayatsu
Copy link
Owner

(設定画面を作るのが少々手間。。)

設定画面は #33 をやるときに作るかもしれませんが、直近では他に必要なさそうですね。

README 等に断りをいれてリリースしてしまうのも (お遊び的なアプリなので) 許されるかもしれませんがどうでしょう?

そうですね、隠し機能的なノリでいきましょうか :)
リポジトリののREADMEには乗っているけど、storeの説明には載っていないぐらいのバランスでよさそうです。

寿司ゆき

fukayatsu added a commit that referenced this pull request May 28, 2014
@fukayatsu fukayatsu merged commit 7fb8f4f into fukayatsu:master May 28, 2014
@fukayatsu
Copy link
Owner

storeにuploadしたので1時間以内に公開されると思います〜

今年「ありがとう」を言いたいこと | ガールズちゃんねる - Girls ...

@shao1555
Copy link
Contributor Author

マージありがとうございます。
寿司ゆき

せっかくリリースしていただいたところ恐縮ですが、サムネだと小さくてちょっと違和感あるので、別の pull-req でなおしてもよいでしょうか。

@fukayatsu
Copy link
Owner

@shao1555 はい、お待ちしております:)

@shao1555 shao1555 deleted the features/google-image-search branch May 29, 2014 12:38
@shao1555
Copy link
Contributor Author

shao1555 commented Dec 3, 2015

ついにお亡くなりになってしまいました

寿司ゆき:sorry

@fukayatsu
Copy link
Owner

R.I.P.

寿司ゆき:see_you

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

Successfully merging this pull request may close these issues.

2 participants