Skip to content

Conversation

@pkfec
Copy link
Contributor

@pkfec pkfec commented Jan 8, 2026

Summary (required)

This PR adds two new celery cron jobs :

  1. First one will publish/unpublish rulemakings from elasticsearch service.
  2. Second cron job send slack message/alert at the end of the day, with list of rulemakings that were modified during that day.

Required reviewers

2-3 developers

Impacted areas of the application

  • api, elasticsearch service and celery

How to test

@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

❌ Patch coverage is 3.38983% with 57 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.53%. Comparing base (fcad303) to head (33fcda0).

Files with missing lines Patch % Lines
webservices/tasks/legal_docs.py 0.00% 41 Missing ⚠️
webservices/legal/rulemaking_docs/rulemaking.py 11.11% 16 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #6456      +/-   ##
===========================================
- Coverage    86.97%   86.53%   -0.45%     
===========================================
  Files           98       98              
  Lines         9999    10051      +52     
===========================================
+ Hits          8697     8698       +1     
- Misses        1302     1353      +51     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pkfec pkfec self-assigned this Jan 8, 2026
@pkfec pkfec moved this to 🏗 In Progress in Website project Jan 8, 2026
@pkfec pkfec linked an issue Jan 8, 2026 that may be closed by this pull request
3 tasks
@pkfec pkfec marked this pull request as draft January 9, 2026 02:42
@pkfec pkfec force-pushed the feature/6436-celery-task-rulemakings branch from 33fcda0 to ee6531f Compare January 20, 2026 23:44
@pkfec pkfec closed this Jan 21, 2026
@github-project-automation github-project-automation bot moved this from 🏗 In Progress to ✅ Done in Website project Jan 21, 2026
@pkfec
Copy link
Contributor Author

pkfec commented Jan 23, 2026

closed in favor of pr #6473

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Add a celery task to upload rulemakings to elasticsearch

2 participants