search

masachika-kamada / ChatGPT-Ctrl-Enter-Sender

ChatGPT用Chrome拡張機能
https://chrome.google.com/webstore/detail/chatgpt-ctrl%20enter-sender/gbncgdhklmnckojlibfhdadpfbcdbnch
49 stars 15 forks source link

Firefox版の拡張機能の作成予定について #20

Closed Aniny21 closed 1 year ago

Aniny21 commented 1 year ago

こんにちは、素晴らしい拡張機能を開発してくださりありがとうございます。

私はFirefoxを主要ブラウザとして使用しているため、この拡張機能のFirefox版があれば非常にありがたいのですが、Firefox版の開発の予定はありますか？

masachika-kamada commented 1 year ago

@Aniny21 ご興味持っていただきありがとうございます。 Firefox でのリリースの予定はありませんでしたが、こちらで検討してみます。 リリースする際にはまたご連絡いたします。 引き続きよろしくお願いいたします。

Aniny21 commented 1 year ago

迅速なご返信ありがとうございます。

この拡張機能のFirefox版が楽しみで待ちきれなかったため、個人的にテストしてみましたところ、FirefoxのManifest V3対応状況が悪いためV2に落とす必要がありましたが、いくつかのAPIを変更し似たような動作を再現することができました。

差し出がましいとは存じますが、もし詳しい情報が必要でしたらお知らせください。

拡張機能の発展と多くのユーザーへの提供を心から応援しております。

masachika-kamada commented 1 year ago

@Aniny21 Firefox 対応に向けた情報のご提供、誠にありがとうございます。

https://github.com/masachika-kamada/ChatGPT-Ctrl-Enter-Sender/tree/firefox-dev こちらのブランチを Firefox 向けの拡張機能開発ブランチとして公開いたしました。 私自身、Firefox の拡張機能を作成したことがなく、Firefox への対応のためのコード修正に時間を要してしまうため、大変恐縮ですがこのブランチに向けて Pull Request を出していただけますと幸いです。

私も @Aniny21 様と同じく多くのユーザーへの利便性の向上が最重要だと考えておりますので、@Aniny21 様ご自身で ChatGPT Ctrl+Enter Sender を Firefox 対応させたものを公開していただいても構いません。

貴重なご意見いただきありがとうございます。

Aniny21 commented 1 year ago

firefox-dev ブランチを拝見させていただきました。 Manifestの設定など私が以前進めていた変更とほぼ同じであったため、編集の必要がありませんでしたが、APIの違いからか一部で TypeError が発生していたため、その部分の修正に関するPull Requestを送らせていただきました。(このままだと、サイトごとの拡張機能オンオフができないため、もう一度Pull Requestを送らせていただくことになると思います。)

ご確認のほどよろしくお願いいたします。

GitHubに関してはまだ初心者の段階ですので、操作ミスや不足している部分が見受けられるかもしれません。 そのような点でご迷惑をおかけすることがあれば、心よりお詫び申し上げます。

masachika-kamada commented 1 year ago

@Aniny21 Pull Request 誠にありがとうございます。マージいたしました。 私も data が未定義になることに関して Firefox では chrome ではなく browser に置き換える必要があるとの記載を見つけたため、私も検証を進めてみます。 引き続きよろしくお願いいたします。

Aniny21 commented 1 year ago

マージありがとうございます。 chromeとbrowserについてはcallbackかPromiseかなど細かい挙動の違いがあるようなので、storage系の処理はかなり書き換える必要があるかもしれません。 引き続き検証させていただきます。

Aniny21 commented 1 year ago

いくつかの変更を加え、とりあえずFirefox版でChrome版と同じ動作を再現することができました。 Pull Requestのご確認をよろしくお願いいたします。

masachika-kamada commented 1 year ago

コード修正ありがとうございます。 Firefox にて Chrome と同じ動作を確認できたためマージいたしました。 動作に関して問題ないと考えられるためリリース作業を進めたいと思いますがいかがでしょうか。 また、README にコントリビュータとしてアカウント名を掲載してもよろしいでしょうか。

Aniny21 commented 1 year ago

ご連絡いただき、ありがとうございます。 リリース作業の進行について、問題ございません。何か私にお手伝いできることがございましたら、ご遠慮なくお申し付けください。

また、README に私のアカウント名をコントリビュータとして記載していただくことを提案していただき、大変光栄に思います。是非ともお願い申し上げます。

masachika-kamada commented 1 year ago

リリースの申請が完了いたしました。 審査が完了すると以下のページからダウンロードできるようになるとのことでした。 https://addons.mozilla.org/ja/firefox/addon/chatgpt-ctrl-enter-sender/ お力添えいただき誠にありがとうございました。 image

Aniny21 commented 1 year ago

リリースの申請が完了されたとのこと、ご報告いただきありがとうございます。

なお、失念していた点が一つございましたので、補足させていただきます。 現在、manifest.jsonのidは一時的にaddon_id_temp@example.comと設定されています。このままでも問題はないと思われますが、「アドオン名@作者名」のように意味のあるIDに変更していただけるとなお良いと思われます。

審査が通り、アドオンが使えるようになるのを心待ちにしております。

masachika-kamada commented 1 year ago

@Aniny21 アドオンの審査が通りダウロードが可能となりました！ issue の作成から実装までしていただき、誠にありがとうございました。

是非今後ともよろしくお願いいたします。