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): 削除して編集で引用ありを消せない #15249

Merged
merged 5 commits into from
Jan 14, 2025

Conversation

tai-cha
Copy link
Contributor

@tai-cha tai-cha commented Jan 10, 2025

What

削除して編集で引用を消すためにprops.renoteを直接利用せずいったん保管して消せるように

Why

Fix #14476
Close #15245

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

Copy link

codecov bot commented Jan 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 20.06%. Comparing base (d082a1d) to head (194e16e).
Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #15249      +/-   ##
===========================================
+ Coverage    19.94%   20.06%   +0.12%     
===========================================
  Files          729      729              
  Lines       104055   104056       +1     
  Branches      1200     1198       -2     
===========================================
+ Hits         20749    20876     +127     
+ Misses       82753    82628     -125     
+ Partials       553      552       -1     

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

Copy link
Member

@Sayamame-beans Sayamame-beans left a comment

Choose a reason for hiding this comment

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

私が見た限り/試した限りでは良さそうです! ありがとうございます!

@kakkokari-gtyih
Copy link
Contributor

renoteTargetNoteとか

@tai-cha
Copy link
Contributor Author

tai-cha commented Jan 12, 2025

@Sayamame-beans changelog追加漏れの追加、非常に助かります!

ところで

  • 代わりに「削除して編集」時の投稿フォームにて引用対象のノートの内容が表示されなくなります。

これってそうなってましたっけ?

@Sayamame-beans
Copy link
Member

ア!
そうならないように実装してもらってたのに書き直し忘れました…
あとで直します…

CHANGELOG.md Outdated Show resolved Hide resolved
@tai-cha
Copy link
Contributor Author

tai-cha commented Jan 12, 2025

ア!
そうならないように実装してもらってたのに書き直し忘れました…
あとで直します…

やっておきました 👍

@Sayamame-beans
Copy link
Member

失礼しました🙏

@mi-gh-maintainer mi-gh-maintainer bot merged commit dd6743d into misskey-dev:develop Jan 14, 2025
8 of 9 checks passed
Copy link

Thank you 🙏

Ruruke pushed a commit to Ruruke/misskey that referenced this pull request Jan 14, 2025
* Fix(frontend): 削除して編集で引用ありを消せない

* docs(changelog): update CHANGELOG.md

* rename noteToRenote -> renoteTargetNote

with type fix

* Update Changelog

---------

Co-authored-by: Sayamame-beans <61457993+Sayamame-beans@users.noreply.github.com>
Co-authored-by: syuilo <4439005+syuilo@users.noreply.github.com>
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
Development

Successfully merging this pull request may close these issues.

「削除して編集」で引用として添付したノートを外せない
4 participants