Skip to content

Conversation

@angelampcosta
Copy link
Contributor

Summary

Adds update job name option.

Documentation checklist

@cloudflare-workers-and-pages
Copy link

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 34d6b82
Status: ✅  Deploy successful!
Preview URL: https://db837eae.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://angela-update-job-name-api.cloudflare-docs-7ou.pages.dev

View logs

@github-actions
Copy link
Contributor

@angelampcosta angelampcosta merged commit c97092c into production Dec 18, 2024
13 checks passed
@angelampcosta angelampcosta deleted the angela/update-job-name-api branch December 18, 2024 11:06
| `GET` | Retrieve all jobs for a dataset | [Documentation](/api/resources/logpush/subresources/datasets/subresources/jobs/methods/get/) |
| `GET` | Retrieve all available fields for a dataset | [Documentation](/api/resources/logpush/subresources/datasets/subresources/fields/methods/get/) |
| `PUT` | Update job | [Documentation](/api/resources/logpush/subresources/jobs/methods/update/) |
| `PUT` | Update job name. | [Documentation](/api/resources/logpush/subresources/jobs/methods/update/) |
Copy link
Contributor

Choose a reason for hiding this comment

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

This is not a separate endpoint, and I don't think it should have been added like this.
We just made name field to be updatable in Update job endpoint.

It would be more appropriate to add a section below to describe briefly what this field is for (mainly for a user to set a descriptive name for a job to make it easier to find / identify their jobs).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @soheiokamoto I've opened another PR to address your comment #18857

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants