fix: allow moderators see role assigned users; fix #13301

[Hoto-Cocoa]

What

This PR changes /admin/roles/users API can be access by moderators, which reported as bug in #13301.

Why

See #13301 to more details.

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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (339acd2) 79.11% compared to head (a7f13de) 79.23%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #13315      +/-   ##
==========================================
+ Coverage   79.11%   79.23%   +0.12%     
==========================================
  Files         951      952       +1     
  Lines      104319   104383      +64     
  Branches     8359     8385      +26     
==========================================
+ Hits        82529    82705     +176     
+ Misses      21790    21678     -112     

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

[github-actions]

このPRによるapi.jsonの差分

差分はこちら

Get diff files from Workflow Page

[Hoto-Cocoa]

I've done test.

The only problem is moderators can see all roles, not only roles that can be assigned by moderators.

[syuilo]

🙏🏻

[anatawa12]

これ間違ってmasterにマージしちゃってるのでdevelopにリベースしてmasterからdropしたほうがいいかも

[Hoto-Cocoa]

これ間違ってmasterにマージしちゃってるのでdevelopにリベースしてmasterからdropしたほうがいいかも

会社での習慣通りマスターにPRを送ってしまいました。 私がしなければならないことがありますか？

[anatawa12]

Hotoさんがするべきことはないと思います。

[Hoto-Cocoa]

Hotoさんがするべきことはないと思います。

確認しました。 ありがとうございます。
貢献したという内訳が残っているのが良いので、これだけうまくいけばどうなってもいいです。

[anatawa12]

貢献したという内訳が残っているのが良いので、これだけうまくいけばどうなってもいいです。

rebaseするときにはAuthorとして残ると思われます。

(gitでは変更の作者であるauthorとcommit作った人であるcommitterが独立して記録されます。)

追記: 作業者はAuthorが残るようにしてください。git cherry-pickかgit rebaseを用いると問題なく行われます。