Skip to content

Commit

Permalink
fix PATCH job definitions active
Browse files Browse the repository at this point in the history
  • Loading branch information
dlqqq committed Oct 7, 2022
1 parent a842625 commit 4b00854
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
6 changes: 5 additions & 1 deletion jupyter_scheduler/handlers.py
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,11 @@ async def post(self):
@tornado.web.authenticated
async def patch(self, job_definition_id):
payload = self.get_json_body()
await ensure_async(self.scheduler.update_job_definition(UpdateJobDefinition(**payload)))
await ensure_async(
self.scheduler.update_job_definition(
UpdateJobDefinition(**payload, job_definition_id=job_definition_id)
)
)
self.set_status(204)
self.finish()

Expand Down
1 change: 1 addition & 0 deletions jupyter_scheduler/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -195,6 +195,7 @@ class UpdateJobDefinition(BaseModel):
url: Optional[str] = None
timezone: Optional[str] = None # Should be a timezone e.g., US/Eastern, Asia/Kolkata
output_filename_template: Optional[str] = OUTPUT_FILENAME_TEMPLATE
active: bool


class ListJobDefinitionsQuery(BaseModel):
Expand Down

0 comments on commit 4b00854

Please sign in to comment.