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

リマインダー機能の改善 #92

Closed
wants to merge 53 commits into from

Conversation

na2na-p
Copy link
Contributor

@na2na-p na2na-p commented Apr 14, 2022

これは何

タイトルの通り。
大きく3つの変更点があります。

renote対象が消されていた場合にリマインダー解除を行う機能

#89 に関係しそう。
「やった」「やめる」でリマインダー解除する場所と同じように書きました。

何が変わるの

リマインダーで、todoしてるノートが消されたら自動的にリマインドも解除する。

リマインド対象のノートのvisibilityがfollowersのときにtodoを受け付けないようにした

todoをお願いしたノートのvisibilityが、followersとなっていた時に🆗とはなる。が、引用RNができなくてただ覚えているだけになっているので、それだったら最初から受け付けないようにすればいいんじゃない?となり書きました。

何が変わるの

  • msg.visibilityで公開範囲が取れるようになる
  • リマインダーで、todoしてるノートのvisibilityが、followersだったら受け付けなくなる
  • 受け付けられなかったときにNGのリアクションを返すようになる

他人のリマインダーを操作できる)バグの修正

「やった」発言をしたユーザーidと、リマインド登録したユーザーidを比較し、一致しなければ画像の一番下のように返すように変更し、リマインダー解除しないようにした
画像

@na2na-p
Copy link
Contributor Author

na2na-p commented Apr 14, 2022

ごめんなさいどこかで事故った。。。

@na2na-p na2na-p closed this Apr 14, 2022
@na2na-p na2na-p deleted the featReminderFix branch April 14, 2022 13:36
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.

1 participant