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(frontend): Recieve Unrenote on streaming #12079

Merged
merged 3 commits into from
Oct 19, 2023

Conversation

atsu1125
Copy link
Contributor

@atsu1125 atsu1125 commented Oct 19, 2023

表示しているリノートがリノート解除されたらストリーミングで受信してすぐに消えるようにする

What

useNoteCaptureで見てsrもしくはsしてるのがappearNoteのみで
ほとんどはこれで問題がないが
リノート解除の場合はpureNotesしていないと受信して反映ができない
そのためappearNotepureNoteが違う場合のみsする
おそらくpureNote対象に流れてくるのはdeletedのみなはずだが
クライアントの表示件数のみストリームに接続するので許容してほしい

Why

Resolve #12078

Additional info (optional)

ローカル環境とテスト環境で確認済み

📌 Environment

Misskey v12.119.2
Misskey 2023.10.2-beta.2

💻 Frontend

  • Model and OS of the device(s):
    macOS Sonoma
    iOS 15.7.1
  • Browser:
    Firefox 115.3.1esr (64-bit)
    Firefox Focus 118.0
  • Server URL:
    Our Misskey Test Environment
  • Misskey:
    Misskey 12.119.2
    Misskey 2023.10.2-beta.2

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

表示しているリノートがリノート解除されたらストリーミングで受信してすぐに消えるようにする
@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Oct 19, 2023
@codecov
Copy link

codecov bot commented Oct 19, 2023

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (721cbe0) 79.21% compared to head (bb775d2) 79.03%.

❗ Current head bb775d2 differs from pull request most recent head dadf205. Consider uploading reports for the commit dadf205 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #12079      +/-   ##
===========================================
- Coverage    79.21%   79.03%   -0.18%     
===========================================
  Files          931      175     -756     
  Lines        99550    22745   -76805     
  Branches      8104      486    -7618     
===========================================
- Hits         78859    17977   -60882     
+ Misses       20691     4768   -15923     
Files Coverage Δ
packages/frontend/src/components/MkNote.vue 100.00% <100.00%> (ø)
packages/frontend/src/scripts/use-note-capture.ts 9.48% <0.00%> (-0.26%) ⬇️

... and 756 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@syuilo syuilo merged commit 7b36122 into misskey-dev:develop Oct 19, 2023
15 checks passed
@syuilo
Copy link
Member

syuilo commented Oct 19, 2023

👍🏻

kanarikanaru pushed a commit to kanarikanaru/misskey that referenced this pull request Nov 2, 2023
* fix(frontend): Recieve Unrenote

表示しているリノートがリノート解除されたらストリーミングで受信してすぐに消えるようにする

* fix(frontend): Recieve Unrenote lint fixing

* fix(frontend): Recieve Unrenote Decapture

Decapture忘れてたー
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot Take back Renote in real-time
2 participants