Skip to content

Commit

Permalink
review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
M-Kusumgar committed Aug 16, 2024
1 parent 5e533b6 commit fca17c8
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 5 deletions.
1 change: 1 addition & 0 deletions .gitattributes
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
R/porcelain.R linguist-generated=true
4 changes: 2 additions & 2 deletions R/api.R
Original file line number Diff line number Diff line change
Expand Up @@ -92,11 +92,11 @@ report_parameters <- function(root, ref, name) {
##' body data :: json(report_run_request)
submit_report_run <- function(root, queue, data) {
data <- jsonlite::parse_json(data)
job_id <- queue$submit(
task_id <- queue$submit(
data$name,
branch = data$branch,
ref = data$hash,
parameters = data$parameters
)
list(job_id = scalar(job_id))
list(taskId = scalar(task_id))
}
4 changes: 3 additions & 1 deletion inst/schema/report_run_request.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,9 @@
{
"type": "object",
"properties": {},
"additionalProperties": true
"additionalProperties": {
"type": ["boolean", "integer", "number", "string"]
}
}
]
}
Expand Down
4 changes: 2 additions & 2 deletions inst/schema/report_run_response.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@
"$schema": "http://json-schema.org/draft-07/schema#",
"type": "object",
"properties": {
"job_id": {
"taskId": {
"type": "string"
}
},
"required": ["job_id"],
"required": ["taskId"],
"additionalProperties": false
}

0 comments on commit fca17c8

Please sign in to comment.