minvws / nl-kat-coordination

Repo nl-kat-coordination for minvws
European Union Public License 1.2
126 stars 57 forks source link

Edit report recipe #3687

Open madelondohmen opened 1 week ago

madelondohmen commented 1 week ago

About this feature

Detailed description

It should be possible to update the report recipe and the schedule (cron-expression).

Specifications

The implementation should include…

Design

Figma link

The button to edit should be here (but should be "Edit report recipe"): https://www.figma.com/design/ASfstypqSx22Bw7XDbVEdo/%E2%9C%85-KAT-%7C-Approved-%7C-Reports-Page?node-id=2308-2773&node-type=frame

Implementation

After saving the ReportRecipe, the scheduler needs to be updated as well (PATCH /schedules/{schedule_id})

underdarknl commented 6 days ago

The scheduler only needs to be patched when the cron is changed, correct?

jpbruinsslot commented 1 day ago

The scheduler only needs to be patched when the cron is changed, correct?

Correct