DalgoT4D / DDP_backend

Django app for the DDP platform
GNU Affero General Public License v3.0
12 stars 41 forks source link

Schedule large connections jobs #861

Closed Ishankoradia closed 1 month ago

Ishankoradia commented 2 months ago

Feature https://github.com/DalgoT4D/DDP_backend/issues/860

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 24.80620% with 97 lines in your changes missing coverage. Please review.

Project coverage is 73.76%. Comparing base (f1ec071) to head (26f89cb).

Files with missing lines Patch % Lines
ddpui/ddpairbyte/airbytehelpers.py 9.52% 76 Missing :warning:
ddpui/ddpprefect/prefect_service.py 17.64% 14 Missing :warning:
ddpui/utils/helpers.py 36.36% 7 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #861 +/- ## ========================================== - Coverage 74.55% 73.76% -0.79% ========================================== Files 194 195 +1 Lines 7966 8085 +119 ========================================== + Hits 5939 5964 +25 - Misses 2027 2121 +94 ``` | [Flag](https://app.codecov.io/gh/DalgoT4D/DDP_backend/pull/861/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=DalgoT4D) | Coverage Δ | | |---|---|---| | [](https://app.codecov.io/gh/DalgoT4D/DDP_backend/pull/861/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=DalgoT4D) | `73.76% <24.80%> (-0.79%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=DalgoT4D#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Ishankoradia commented 1 month ago

Its ready for review @fatchat For reset jobs when clicked on the 3 dots menu on a connection, i am not sure where to show that your reset is scheduled for the weekend. Corresponding frontend #861

Ishankoradia commented 1 month ago