Firefox版の拡張機能の作成予定について
Aniny21 opened this issue · comments
こんにちは、素晴らしい拡張機能を開発してくださりありがとうございます。
私はFirefoxを主要ブラウザとして使用しているため、この拡張機能のFirefox版があれば非常にありがたいのですが、Firefox版の開発の予定はありますか？
@Aniny21
ご興味持っていただきありがとうございます。
Firefox でのリリースの予定はありませんでしたが、こちらで検討してみます。
リリースする際にはまたご連絡いたします。
引き続きよろしくお願いいたします。
迅速なご返信ありがとうございます。
この拡張機能のFirefox版が楽しみで待ちきれなかったため、個人的にテストしてみましたところ、FirefoxのManifest V3対応状況が悪いためV2に落とす必要がありましたが、いくつかのAPIを変更し似たような動作を再現することができました。
差し出がましいとは存じますが、もし詳しい情報が必要でしたらお知らせください。
拡張機能の発展と多くのユーザーへの提供を心から応援しております。
@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 対応させたものを公開していただいても構いません。
貴重なご意見いただきありがとうございます。
firefox-dev ブランチを拝見させていただきました。
Manifestの設定など私が以前進めていた変更とほぼ同じであったため、編集の必要がありませんでしたが、APIの違いからか一部で TypeError が発生していたため、その部分の修正に関するPull Requestを送らせていただきました。(このままだと、サイトごとの拡張機能オンオフができないため、もう一度Pull Requestを送らせていただくことになると思います。)
ご確認のほどよろしくお願いいたします。
GitHubに関してはまだ初心者の段階ですので、操作ミスや不足している部分が見受けられるかもしれません。
そのような点でご迷惑をおかけすることがあれば、心よりお詫び申し上げます。
@Aniny21
Pull Request 誠にありがとうございます。マージいたしました。
私も data が未定義になることに関して Firefox では chrome ではなく browser に置き換える必要があるとの記載を見つけたため、私も検証を進めてみます。
引き続きよろしくお願いいたします。
マージありがとうございます。
chromeとbrowserについてはcallbackかPromiseかなど細かい挙動の違いがあるようなので、storage系の処理はかなり書き換える必要があるかもしれません。
引き続き検証させていただきます。
いくつかの変更を加え、とりあえずFirefox版でChrome版と同じ動作を再現することができました。
Pull Requestのご確認をよろしくお願いいたします。
コード修正ありがとうございます。
Firefox にて Chrome と同じ動作を確認できたためマージいたしました。
動作に関して問題ないと考えられるためリリース作業を進めたいと思いますがいかがでしょうか。
また、README にコントリビュータとしてアカウント名を掲載してもよろしいでしょうか。
ご連絡いただき、ありがとうございます。
リリース作業の進行について、問題ございません。何か私にお手伝いできることがございましたら、ご遠慮なくお申し付けください。
また、README に私のアカウント名をコントリビュータとして記載していただくことを提案していただき、大変光栄に思います。是非ともお願い申し上げます。
リリースの申請が完了いたしました。
審査が完了すると以下のページからダウンロードできるようになるとのことでした。
https://addons.mozilla.org/ja/firefox/addon/chatgpt-ctrl-enter-sender/
お力添えいただき誠にありがとうございました。
リリースの申請が完了されたとのこと、ご報告いただきありがとうございます。
なお、失念していた点が一つございましたので、補足させていただきます。
現在、manifest.jsonのidは一時的にaddon_id_temp@example.comと設定されています。このままでも問題はないと思われますが、「アドオン名@作者名」のように意味のあるIDに変更していただけるとなお良いと思われます。
審査が通り、アドオンが使えるようになるのを心待ちにしております。
@Aniny21
アドオンの審査が通りダウロードが可能となりました！
issue の作成から実装までしていただき、誠にありがとうございました。
是非今後ともよろしくお願いいたします。