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.

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

Add env variables to toggle summarization tasks #1809

Merged
merged 4 commits into from
Jan 18, 2024

Conversation

amjithtitus09
Copy link
Contributor

@amjithtitus09 amjithtitus09 commented Jan 6, 2024

Proposed Changes

  • Added env variables to disable each Summarization task.

Associated Issue

Merge Checklist

  • Tests added/fixed
  • Update docs in /docs
  • Linting Complete
  • Any other necessary step

Only PR's with test cases included and passing lint and test pipelines will be reviewed

@coronasafe/care-backend-maintainers @coronasafe/care-backend-admins

config/settings/base.py Outdated Show resolved Hide resolved
@amjithtitus09
Copy link
Contributor Author

@sainak Made the changes and added Docs

@sainak
Copy link
Member

sainak commented Jan 6, 2024

@amjithtitus09 lint check is failing, pass the changed files through pre-commit

@sainak
Copy link
Member

sainak commented Jan 6, 2024

@sainak sainak changed the title Summarization Tasks Disable - Added env variables Add env variables to toggle summarization tasks Jan 6, 2024
@coronasafe coronasafe deleted a comment from codecov bot Jan 8, 2024
Copy link

codecov bot commented Jan 8, 2024

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (ef73d93) 61.04% compared to head (1560692) 61.02%.

Files Patch % Lines
care/facility/tasks/__init__.py 54.54% 0 Missing and 5 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1809      +/-   ##
==========================================
- Coverage   61.04%   61.02%   -0.03%     
==========================================
  Files         208      208              
  Lines       11526    11532       +6     
  Branches     1656     1661       +5     
==========================================
+ Hits         7036     7037       +1     
  Misses       4218     4218              
- Partials      272      277       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@vigneshhari vigneshhari merged commit 7ba0905 into coronasafe:master Jan 18, 2024
5 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add environment variable to disable summarization cron tasks
5 participants