#17 「いいね解除」された時に投稿が増えるバグ、及び、付随するバグの修正

Scalone
senooken scala 1 commity/ów z tak4/main do gnusocialjp/main 9 miesięcy temu
tak4 skomentował 9 miesięcy temu

#10 の残りのバグを修正しました。
mastodonとの会話で増幅するバグについては、再現方法が分らないので未確認ですが、今回の修正で直っている可能性があります。
また、リモートとの「いいね解除」の通信がうまく行かない事があったので、そちらも修正しました。

  • 外部サーバーから「いいね解除」された時の増幅を防止する。
    Activitypub_notice.php
  • 「いいね解除」した事を外部サーバーに通知する。
    Activitypub_like.php
  • 外部サーバーからの「いいね解除」をローカルDBに反映する。
    Activitypub_like.php

( #14 の反映はできていないので、もし #14 を先にマージされるのであれば、その後で修正後、再度プルリクエストします。)

#10 の残りのバグを修正しました。 mastodonとの会話で増幅するバグについては、再現方法が分らないので未確認ですが、今回の修正で直っている可能性があります。 また、リモートとの「いいね解除」の通信がうまく行かない事があったので、そちらも修正しました。 - 外部サーバーから「いいね解除」された時の増幅を防止する。 Activitypub_notice.php - 「いいね解除」した事を外部サーバーに通知する。 Activitypub_like.php - 外部サーバーからの「いいね解除」をローカルDBに反映する。 Activitypub_like.php ( #14 の反映はできていないので、もし #14 を先にマージされるのであれば、その後で修正後、再度プルリクエストします。)
senooken skomentował 9 miesięcy temu
Właściciel

対応ありがとうございます!内容、修正量的に、こちらを先にマージしたいです。PHP 8対応は確認や修正に時間がかかると思います。 他の作業との兼ね合いで、確認に時間がかかるかもしれませんので、お待ちください…

対応ありがとうございます!内容、修正量的に、こちらを先にマージしたいです。PHP 8対応は確認や修正に時間がかかると思います。 他の作業との兼ね合いで、確認に時間がかかるかもしれませんので、お待ちください…
tak4 skomentował 9 miesięcy temu
Autor

承知いたしました。よろしくお願いいたします。

承知いたしました。よろしくお願いいたします。

tak4さんの変更は、PHP8対応をマージ前にも、問題無しでこれをマージ出来るはずです

tak4さんの変更は、PHP8対応をマージ前にも、問題無しでこれをマージ出来るはずです
senooken skomentował 9 miesięcy temu
Właściciel

suwako、コメントありがとうございます。ピンポイントの修正なのでおそらく大丈夫だろうと私も思っています。現状この2個しかマージ予定ないので、修正量の少ないこちらをやはり先にマージします。

suwako、コメントありがとうございます。ピンポイントの修正なのでおそらく大丈夫だろうと私も思っています。現状この2個しかマージ予定ないので、修正量の少ないこちらをやはり先にマージします。
senooken skomentował 9 miesięcy temu
Właściciel

@tak4 修正が入ることで、相手からいいねの解除が来た場合の再送がなくなることを確認できました。

また、再投稿の場合は、二重化はそもそも発生していませんでした。OKです。ありがとうございました。

@tak4 修正が入ることで、相手からいいねの解除が来た場合の再送がなくなることを確認できました。 また、再投稿の場合は、二重化はそもそも発生していませんでした。OKです。ありがとうございました。
Ten pull request został pomyślnie scalony!
Zaloguj się, aby dołączyć do tej rozmowy.
Brak kamienia milowego
Brak przypisania
3 uczestników
Ładowanie...
Anuluj
Zapisz
Nie ma jeszcze treści.