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

Fix: [Server][Channel]バックエンドでmirakurunを選択時にKeyErrorで起動しない問題の修正 #42

Merged
merged 1 commit into from
May 21, 2023

Conversation

YSHKatN
Copy link
Contributor

@YSHKatN YSHKatN commented May 21, 2023

updateFromMirakurunのチャンネルID選択時のキーを修正

変更の種類

  • 不具合の修正
  • 新しい機能の追加
  • 改善・リファクタリング(機能は追加されないが、動作やコードを改善する破壊的でない変更)

チェックリスト:

  • 開発資料データベース設計 は読みましたか?
    • もともと自分用のメモですが、このプロジェクトの開発方針や設計などが記載されています。開発時の参考にしてください。
  • Git のコミットメッセージは開発資料に記載のフォーマットになっていますか?(重要)
    • このプロジェクト上のコミットメッセージは、基本的に全てこのフォーマットに従って記述されています(文字数は問いません)。
    • 正しいフォーマットになっていない場合、force push で必ずコミットメッセージを変更してから送信してください。
  • このプロジェクトのコーディング規約に従ったコードになっていますか?
    • コーディング規約は開発資料に記載されています。
    • そもそもコーディング規約と言えるほど大層なものではありませんが、一度目を通しておいてください。
  • プルリクエストは master ブランチに送信されていますか?
    • release ブランチはリリースした時にしか更新されません。必ず master ブランチに送信してください。

説明

channel_idへ代入時、serviceに指定するkey名がonid,sidとなっているのを、networkId,serviceIdへ変更。

動機とコンテキスト

mirakurunをバックエンドとして選択している場合、起動時にKeyErrorとなり起動できない問題が解決。

@tsukumijima
Copy link
Owner

プルリクエストをありがとうございます!
普段 EDCB バックエンドで開発しているのですが、リファクタリングした際にコピペミスしてしまったみたいです。
本当に助かりました!ありがとうございました…!!

@tsukumijima tsukumijima merged commit cee534c into tsukumijima:master May 21, 2023
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.

None yet

2 participants