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(backend): muteがapiからのuser list timeline取得で機能しない #11480

Conversation

anatawa12
Copy link
Member

What

muteがapiからのuser list timeline取得で機能しないので修正しました。

Why

Channel Streamからだとmuteが機能するのにこっちからだと機能しないのは不自然だと思ったため

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/backend Server side specific issue/PR label Aug 7, 2023
@codecov
Copy link

codecov bot commented Aug 7, 2023

Codecov Report

Merging #11480 (6063508) into develop (db5046e) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop   #11480      +/-   ##
===========================================
- Coverage    78.64%   78.61%   -0.04%     
===========================================
  Files          921      920       -1     
  Lines        97493    97392     -101     
  Branches      7696     7694       -2     
===========================================
- Hits         76670    76560     -110     
- Misses       20823    20832       +9     
Files Changed Coverage Δ
...c/server/api/endpoints/notes/user-list-timeline.ts 54.79% <0.00%> (-1.55%) ⬇️

... and 3 files with indirect coverage changes

@syuilo syuilo merged commit b26e4dc into misskey-dev:develop Aug 8, 2023
17 checks passed
@syuilo
Copy link
Member

syuilo commented Aug 8, 2023

👍

@anatawa12 anatawa12 deleted the mute-not-performed-for-user-list-timeline branch August 8, 2023 06:03
@anatawa12
Copy link
Member Author

#10443 っぽい?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend Server side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants