Fujita Shu
Fujita Shu
Is this gem still maintained? This PR is left alone for a long time. If there is no problem, please merge this PR, or if the maintainers will want to...
Hello @ctran, the conflict is in `.rubocop_todo.yml`. So I excuted `rubocop --auto-gen-config` and re-generate `.rubocop_todo.yml`. Now this PR is valid. Please merge if there is no problem.
@ctran I cherry-picked the commit in #865 (e874241) to this pull request, and the CI worked fine. So after #865 merged, I will rebase this pull request onto `develop` branch....
Hi @ocarta-l Ok, I will check #865.
@james2m TravisCI is failed in Ruby 2.3 due to this error. ``` $ bundle install --jobs=3 --retry=3 The dependency activerecord-jdbcsqlite3-adapter (~> 1.3.25) will be unused by any of the platforms...
@james2m Sorry for the slow response, too. Thanks you for merging my PR. I am expecting the release of v1.0.0. Thank you.
@goki90210 @kaizumaki この重複は仕方ないと思います.コンポーネントを共通化しても, * 「1つの Card あたり1つのコンポーネント」の原則が崩れる * よって可読性が悪くなる * 片方の Card の内容だけ変更した場合に対応が面倒になる といったデメリットがあるので,重複をなくすだけの目的で共通化するのは好ましくないと思います. SonarQube の指摘を愚直にすべて直すのではなく,参考程度にとどめておく方がよいのではないでしょうか.
@goki90210 > 「処理」をやっているのであれば「処理」を共通化できませんか? なるほど.失礼しました. よくよく読むと, ```typescript TimeBarChart.extends({ computed: { displayInfo() { const { lastDay, lastDayData } = calcDayBeforeRatio({ displayData: this.displayData, dataIndex: 1, }) const formattedLastDay = this.$d(lastDay, 'date') if (this.dataKind...
## component のディレクトリ構成(案) * `page/index.vue` のみで使われている `components/` 以下のファイルを,`components/index` 以下に移動する(`components/flow` と同様) * `components/cards/` 以下のファイルも,`components/index/cards/` に移動する * `components/cards/` で `import` している component(主にグラフ描画用の `XxxxChart`)を `components/index/cards/charts/` に移動する * (2020-08-22 **追記**)`XxxxCard` に対応する component は `XxxxChart`...
## component の使い方問題 例:`components/cards/TestedCasesDetailsCard.vue` 8行目〜 ```vue {{ $t('(注)医療機関が保険適用で行った検査は含まれていない') }} {{ $t( '(注)検査実施人数には、チャーター機帰国者、クルーズ船乗客等は含まれていない' ) }} {{ $t( '(注)速報値として公開するものであり、後日確定データとして修正される場合あり' ) }} ``` `DataView.vue` の実装を考えると,本来は `` ではなく `` を使うべき. このような箇所が `ConfirmedCasesDetailsCard.vue`, `TestedNumberBarChart.vue` にも見られる....