Skip to content

Conversation

@sgratzl
Copy link
Member

@sgratzl sgratzl commented Sep 20, 2021

closes #1010

Prerequisites:

  • Unless it is a hotfix it should be merged against the dev branch
  • Branch is up-to-date with the branch to be merged with, i.e. dev
  • Build is successful
  • Code is cleaned up and formatted

Summary

changes the code to avoid duplicates from the google docs and reduces them to only the necessary elements.

However, atm. two signals are not delivered by the metadata:

image

see also cmu-delphi/delphi-epidata#717 and cmu-delphi/delphi-epidata#716

@sgratzl sgratzl added the chore label Sep 20, 2021
@sgratzl sgratzl requested a review from krivard September 20, 2021 16:08
@sgratzl sgratzl self-assigned this Sep 20, 2021
@netlify
Copy link

netlify bot commented Sep 21, 2021

✔️ Preview link ready!

🔨 Explore the source changes: 6d1d9e8

🔍 Inspect the deploy log: https://app.netlify.com/sites/cmu-delphi-covidcast/deploys/6149fa56e11c0e0008692742

😎 Browse the preview: https://deploy-preview-1012--cmu-delphi-covidcast.netlify.app

@sgratzl sgratzl marked this pull request as ready for review September 21, 2021 14:33
@sgratzl sgratzl mentioned this pull request Sep 21, 2021
4 tasks
@sgratzl sgratzl enabled auto-merge September 21, 2021 16:03
Copy link
Contributor

@krivard krivard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@sgratzl sgratzl merged commit fe440fe into dev Sep 22, 2021
@sgratzl sgratzl deleted the sgratzl/refactor_docs branch September 22, 2021 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove redundant information from signals doc

3 participants