IRC-WIR / ft_irc

0 stars 0 forks source link

12 add privmsg command #84

Closed weijuan82113 closed 10 hours ago

weijuan82113 commented 1 week ago

privmsgのPRです。 チャンネルの送信を確認していない以外は確認済みです、 下記は確認内容とまります。

テストケース

認証

weijuan82113 commented 1 week ago

指摘を修正しました、確認をお願いします

weijuan82113 commented 1 week ago

再度修正しました、ご確認をお願いします。

weijuan82113 commented 2 days ago

既存のコードをマージして下記テストを行いました

テスト

失敗:

weijuan82113 commented 2 days ago

お疲れさまでした。 すみません。TOPICに関する指摘もこちらに記載してしまいました。

混乱を防ぐために、 PrivmsgブランチはTOPICのコードをマージして、 ここでTOPICの修正も行います。

weijuan82113 commented 1 day ago

動作確認で、privmsgはparamsを全部送信し切れなかったことを気づき、その修正を行う。

paphio-kitsuki commented 1 day ago

お疲れさまでした。 良いと思います。 ただ、user_countとtarget_numの都合上、マージはMODEのあとに行ってください。

paphio-kitsuki commented 10 hours ago

コンフリクト修正後、コンパイルエラーを修正しました。 軽く動作確認を行い、問題なくメッセージが届くことを確認しました。