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.

Sign up for GitHub

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

Jump to bottom

AdminのApplicationServiceのメソッド名をByAdminからForAdminに修正する #2181

Closed
1 task
KentaHizume opened this issue Dec 26, 2024 · 0 comments · Fixed by #2186
Closed
1 task

AdminのApplicationServiceのメソッド名をByAdminからForAdminに修正する #2181

KentaHizume opened this issue Dec 26, 2024 · 0 comments · Fixed by #2186
Labels
target: Dressca サンプルアプリケーションDresscaに関係がある
Milestone
v1.0.0

Comments

@KentaHizume
Copy link
Contributor

KentaHizume commented Dec 26, 2024 •
edited
Loading

概要

アプリケーションサービスのメソッド名getCatalogItemsByAdminについて、
前置詞Byは英語的に不自然であるため、ForAdminに修正する。

詳細

調査した限りでは、受動態の動詞の場合はByの後に行為主体が来ることは問題ないが、
能動態の場合、Byの後には手段や目的が来るべきで、行為主体が来るのは不自然という結果だった。

完了条件

  • メソッド名および影響箇所が修正されていること
@KentaHizume KentaHizume added not triaged 精緻化、分析がされていない target: Dressca サンプルアプリケーションDresscaに関係がある and removed not triaged 精緻化、分析がされていない labels Dec 26, 2024
@KentaHizume KentaHizume added this to the v1.0.0 milestone Dec 26, 2024
@KentaHizume KentaHizume linked a pull request Dec 26, 2024 that will close this issue
メソッド名の ByAdmin を ForAdmin に変更 #2186
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
target: Dressca サンプルアプリケーションDresscaに関係がある
Projects
None yet
Milestone
v1.0.0
Development

Successfully merging a pull request may close this issue.

メソッド名の ByAdmin を ForAdmin に変更 AlesInfiny/maris
1 participant
@KentaHizume