Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

「陽性者の属性」card 専用のファイル名を適切な名前に変更する #5276

Closed
nard-tech opened this issue Aug 14, 2020 · 3 comments
Labels
improvement 改善や新機能の要望

Comments

@nard-tech
Copy link
Contributor

nard-tech commented Aug 14, 2020

改善詳細 / Details of Improvement

  • components/DataTable.vue はその名前から様々な card で import されて使用されているかと思いきや, components/cards/ConfirmedCasesAttributesCard.vue でしか使われていない.
    名称から用途を想像できない上,今後 card を追加するときに変な使い方をされるおそれもあるため, card に合わせて components/ComfirmedCasesAttributesTable.vue に改名する.
@nard-tech nard-tech added the improvement 改善や新機能の要望 label Aug 14, 2020
@nard-tech nard-tech changed the title components/DataTable.vue を components/ComfirmedCasesAttributesTable.vue に改名する 「陽性者の属性」card 専用のファイル名を適切な名前に変更する Aug 14, 2020
@kaizumaki
Copy link
Collaborator

@nard-tech ご提案ありがとうございます。コンポーネントのリファクタリングに関することは近いところで #5064 が提案されておりますので、そちらで議論をしてもらうほうがありがたいなと思っています。

@nard-tech
Copy link
Contributor Author

@kaizumaki ありがとうございます.承知しました.#5064 で提案してみますので,この issue は close します.

@nard-tech
Copy link
Contributor Author

cf. #2790 (comment)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
improvement 改善や新機能の要望
Projects
None yet
2 participants