Skip to content

API for Update Application#778

Open
Bleama02 wants to merge 9 commits into
IBM:mainfrom
Bleama02:update-application
Open

API for Update Application#778
Bleama02 wants to merge 9 commits into
IBM:mainfrom
Bleama02:update-application

Conversation

@Bleama02
Copy link
Copy Markdown
Contributor

Adds an endpoint for Update Application based on application-deployment-api-proposal.md, 5.2.4 Update Application.

  • Updates the display name of existing applications
  • Adds error responses

Bleama02 added 2 commits May 20, 2026 12:19
Signed-off-by: MarcusBlea <bleama02@gmail.com>
Signed-off-by: MarcusBlea <bleama02@gmail.com>
@Bleama02 Bleama02 requested a review from mayuka-c May 20, 2026 19:26
Bleama02 added 3 commits May 20, 2026 14:42
Signed-off-by: MarcusBlea <bleama02@gmail.com>
Signed-off-by: MarcusBlea <bleama02@gmail.com>
Signed-off-by: MarcusBlea <bleama02@gmail.com>
Copy link
Copy Markdown
Member

@mayuka-c mayuka-c left a comment

Choose a reason for hiding this comment

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

Changes LGTM,
Can you attach the response body?

@mayuka-c mayuka-c requested a review from Shubhamag12 May 21, 2026 07:57
Copy link
Copy Markdown
Contributor

@Shubhamag12 Shubhamag12 left a comment

Choose a reason for hiding this comment

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

minor nit
rest lgtm

Comment thread ai-services/internal/pkg/catalog/apiserver/router.go Outdated
@Bleama02
Copy link
Copy Markdown
Contributor Author

Changes LGTM, Can you attach the response body?

@mayuka-c I think this is what you mean? Let me know if not.

image

mayuka-c
mayuka-c previously approved these changes May 22, 2026
Copy link
Copy Markdown
Member

@mayuka-c mayuka-c left a comment

Choose a reason for hiding this comment

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

LGTM

Shubhamag12
Shubhamag12 previously approved these changes May 22, 2026
Copy link
Copy Markdown
Contributor

@Shubhamag12 Shubhamag12 left a comment

Choose a reason for hiding this comment

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

lgtm

@Bleama02 Bleama02 dismissed stale reviews from Shubhamag12 and mayuka-c via c6851e6 May 22, 2026 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants