Skip to content

chore(etl): update the models to include the event_id.#2737

Merged
sudip-khanal merged 5 commits into
project/alert-system-workflowfrom
feat/update-etl-pipeline
May 18, 2026
Merged

chore(etl): update the models to include the event_id.#2737
sudip-khanal merged 5 commits into
project/alert-system-workflowfrom
feat/update-etl-pipeline

Conversation

@sandeshit
Copy link
Copy Markdown
Contributor

  • Update the models with event_id and parent_event_id.
  • Migrations
  • Move fetch url to helpers.

Changes

  • Use the new links logic.

  • Update the models with event_id and parent_event_id.

  • Migrations

  • Move fetch url to helpers.

  • Updated/ran unit tests

  • Updated CHANGELOG.md

Copy link
Copy Markdown
Contributor

@ranjan-stha ranjan-stha left a comment

Choose a reason for hiding this comment

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

LGTM. Please check few comments.

Comment thread alert_system/models.py Outdated
Comment thread alert_system/etl/base/extraction.py
Comment thread alert_system/etl/base/extraction.py Outdated
Comment thread alert_system/etl/gdacs_cyclone/transform.py
sandeshit added 3 commits May 18, 2026 16:04
- Update the models with event_id and parent_event_id.
- Migrations
- Move fetch url to helpers.
- Update tasks to attach related events in latest episode only.
- Update the base extraction, transformation and load.
- Updates accordingly in Gdacs Flood and USGS EQ.
@sandeshit sandeshit force-pushed the feat/update-etl-pipeline branch from 80e28c5 to e93178b Compare May 18, 2026 10:21
@sandeshit sandeshit marked this pull request as ready for review May 18, 2026 11:04
@sudip-khanal sudip-khanal merged commit 6d73cdb into project/alert-system-workflow May 18, 2026
3 checks passed
@sudip-khanal sudip-khanal deleted the feat/update-etl-pipeline branch May 18, 2026 11:06
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.

3 participants