Skip to content

[8.17] [Dashboards] Update response on HTTP API should return a 200, not 201 (#217054)#217498

Merged
olapawlus merged 5 commits intoelastic:8.17from
olapawlus:backport/8.17/pr-217054
Apr 11, 2025
Merged

[8.17] [Dashboards] Update response on HTTP API should return a 200, not 201 (#217054)#217498
olapawlus merged 5 commits intoelastic:8.17from
olapawlus:backport/8.17/pr-217054

Conversation

@olapawlus
Copy link
Copy Markdown
Member

Backport

This will backport the following commits from main to 8.17:

Questions ?

Please refer to the Backport tool documentation

…elastic#217054)

## Summary

Changes response from 201 to 200 in the dashboard updateRoute.

Closes: elastic#212673
(cherry picked from commit 3c2b4da)

# Conflicts:
#	src/platform/test/api_integration/apis/dashboards/update_dashboard/main.ts
@olapawlus olapawlus requested a review from kibanamachine as a code owner April 8, 2025 11:49
@olapawlus olapawlus added the backport This PR is a backport of another PR label Apr 8, 2025
@olapawlus olapawlus enabled auto-merge (squash) April 8, 2025 11:49
@olapawlus olapawlus self-assigned this Apr 9, 2025
@olapawlus
Copy link
Copy Markdown
Member Author

@elasticmachine merge upstream

@olapawlus
Copy link
Copy Markdown
Member Author

@elasticmachine merge upstream

@olapawlus olapawlus merged commit a77e876 into elastic:8.17 Apr 11, 2025
8 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

cc @olapawlus

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

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants