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): リノートの判定が甘いのを修正 #14396

Merged
merged 12 commits into from
Aug 17, 2024

Conversation

kakkokari-gtyih
Copy link
Contributor

What

  • get-note-menuのisRenoteの一部において判定が不十分だったのを修正
  • isRenote, getAppearNoteを共通化して判定基準をあわせるように

Why

Fix https://misskey.io/notes/9wueduyg27vy0437

一部の自ノートで削除ボタンが出ないのを修正

Additional info (optional)

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 Aug 12, 2024
Copy link

codecov bot commented Aug 12, 2024

Codecov Report

Attention: Patch coverage is 52.17391% with 11 lines in your changes missing coverage. Please review.

Project coverage is 41.65%. Comparing base (2632204) to head (442cf00).

Files Patch % Lines
packages/frontend/src/scripts/get-note-menu.ts 14.28% 6 Missing ⚠️
...ackages/frontend/src/components/MkNoteDetailed.vue 0.00% 3 Missing ⚠️
packages/frontend/src/scripts/get-appear-note.ts 80.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #14396      +/-   ##
===========================================
+ Coverage    39.91%   41.65%   +1.73%     
===========================================
  Files         1545     1550       +5     
  Lines       190531   196560    +6029     
  Branches      2701     2692       -9     
===========================================
+ Hits         76054    81874    +5820     
- Misses      113916   114123     +207     
- Partials       561      563       +2     

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

@kakkokari-gtyih kakkokari-gtyih added this to the v2024.8.0 milestone Aug 12, 2024
@kakkokari-gtyih
Copy link
Contributor Author

👀

@kakkokari-gtyih
Copy link
Contributor Author

こぬふりくと解消

@@ -0,0 +1,12 @@
import type { Note, Renote } from './entities.js';

export function isRenote(note: Note): note is Renote {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isPureRenoteの方が良さそう

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

型の名前もPureRenoteにしておく?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

する

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@syuilo syuilo merged commit 059eb6d into misskey-dev:develop Aug 17, 2024
21 of 22 checks passed
@syuilo
Copy link
Member

syuilo commented Aug 17, 2024

🙏🏿

@kakkokari-gtyih kakkokari-gtyih deleted the fix-remote-quote-note-cw branch August 17, 2024 02:37
LemonDouble pushed a commit to LemonDouble/misskey that referenced this pull request Aug 19, 2024
* fix(frontend): リノートの判定が甘いのを修正

* fix

* Update Changelog

* fix

* use type assertion

* fix + add comments

* lint

* misskey-jsに移動

* PureRenote -> Renote

* isRenote -> isPureRenote
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 packages/misskey-js
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants