Feature/#49 add announce date

MizukiSonoko commented 4 years ago

👏 解決する issue / Resolved Issues

close #49

⛏ 変更内容 / Details of Changes

DataViewBasicInfoPanel に対し新規に ladText (last acquisite date) を追加
陽性患者の属性 / 陽性患者数 / 検査実施数 / 新型コロナコールセンター相談件数 / 帰国者・接触者相談センター相談件数について集計日の追加

陽性患者の属性

patientsTable.datasets の先頭要素の公開日を表示 070009_fukushima_covid19_patients_** csv内 公表_年月日 カラムの一番最近の値と同じ

陽性患者数

graphData.patients の末尾要素の日時を表示 070009_fukushima_covid19_patients_** csv内 公表_年月日 カラムの最近の値と同じ

検査実施数

inspectionsGraph の末尾要素の日時を表示 070009_fukushima_covid19_test_count_*** csv内 実施_年月日 カラムの最近の値と同じ

新型コロナコールセンター相談件数

graphData.contacts の末尾要素の日時を表示 070009_fukushima_covid19_call_center_*** csv内 受付_年月日 カラムの最近の値と同じ

帰国者・接触者相談センター相談件数

graphData.querents の末尾要素の日時を表示 070009_fukushima_covid19_returnee_contact_call_center_*** csv内 受付_年月日 カラムの最近の値と同じ

📸 スクリーンショット / Screenshots

P.S.

各文言、位置についてまだしっくり来ていない。
DataViewBasicInfoPanel の引数名 lastAcquisiteDate がしっくり来ていない。
前提としてcsvのデータはソート済みであり、末尾が最新日時のデータであるとしている。

MizukiSonoko commented 4 years ago

 GAE Deploy for review1s
##[error]Process completed with exit code 1.
Run gcloud app deploy app.yaml --no-promote -v 56
ERROR: (gcloud.app.deploy) You do not currently have an active account selected.
Please run:

  $ gcloud auth login

to obtain new credentials, or if you have already logged in with a
different account:

  $ gcloud config set account ACCOUNT

to select an already authenticated account to use.
##[error]Process completed with exit code 1.

secretsがNone

      - name: GCP Authenticate
        uses: GoogleCloudPlatform/github-actions/setup-gcloud@master
        with:
          version: '285.0.0'
          project_id: ${{ secrets.PROJECT_ID }}
          service_account_email: ${{ secrets.GCP_SA_EMAIL }}
          service_account_key: ${{ secrets.GCP_SA_KEY }}
          export_default_credentials: true

Run GoogleCloudPlatform/github-actions/setup-gcloud@master
  with:
    version: 285.0.0
    export_default_credentials: true

ひょっとして別RepoからのPRだとsecretが共有されない……？

MizukiSonoko commented 4 years ago

暗号化されたシークレットのワークフロー内での利用 GITHUB_TOKENを除き、フォークしたリポジトリからワークフローがトリガーされた場合、シークレットは runner に渡されません。

https://help.github.com/ja/actions/configuring-and-managing-workflows/creating-and-storing-encrypted-secrets

:cry:

donuzium commented 4 years ago

なるほどー。外部から実行できないとそれはそれでセキュアだなぁと思うけども…。調べてもらってありがとうございます！

donuzium commented 4 years ago

陽性患者の属性について

この間も問題に上がっていましたが、0人だったときの場合の公開日が最新の属性患者の公表日と一致しないんですよね。なのでここはリクエストもないので無くてもいいかな、と思いました。

陽性患者数について

graphData.patients の末尾要素の日時を表示 070009_fukushima_covid19patients** csv内公表_年月日カラムの最近の値と同じ

上段は正しいんですが、下段は正しくは 070009_fukushima_covid19patients** csvのファイル更新日 - 1日（24時間）という処理になっています。これも0人の日の処理をするためです。

「大きく表示された数字がいつのものか？」ということを明確にするというのがこのIssueの目的だと思うので、検査数も同様に、文言も「公表日」のままで良いかなと思います。

表示の位置について

公表日: 4/20 21人というように並ぶのはどうでしょうか？（サイズは今のままで）その方が「○日に✗人」というように見えて、情報を捉えやすくなるような気がします。

donuzium commented 4 years ago

ちょっとこちらでもイメージを作ってみます。

gdg-fukushima / covid19