Skip to content

feat(api)- Change the name and operation of Container-related REST APIs on SettingsService for Dashboard[#299]#302

Merged
youngtaekiim merged 2 commits intoeclipse-pullpiri:mainfrom
akshaylg0314:main
Sep 29, 2025
Merged

feat(api)- Change the name and operation of Container-related REST APIs on SettingsService for Dashboard[#299]#302
youngtaekiim merged 2 commits intoeclipse-pullpiri:mainfrom
akshaylg0314:main

Conversation

@akshaylg0314
Copy link
Contributor

📝 PR Description

feat(api)- Change the name and operation of Container-related REST APIs on SettingsService for Dashboard[https://github.com//issues/299]
🔗 Related Issue
closes->
#299

🧪 Test Method
Restapis and doc is updated with removed apis and new yaml apis

✅ Checklist
[✅] Code conventions are followed
[✅] Tests are added/modified
[✅] Documentation is updated (if necessary)

@akshaylg0314
Copy link
Contributor Author

Hey @daeyoung-jeong-lge @youngtaekiim @leeeunkoo
please review the code changes,I tested the apply_yaml and withdraw_yaml through the curl commands as i mentioned in Readme.md file in settingsservice, it is working fine.
Thanks!!

Copy link
Contributor

@daeyoung-jeong-lge daeyoung-jeong-lge left a comment

Choose a reason for hiding this comment

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

Looks good.

@daeyoung-jeong-lge
Copy link
Contributor

@akshaylg0314 You don't need to care about the fmt check fail in this PR. Please just make sure to check fmt in the next PR. Thanks.
@leeeunkoo @youngtaekiim I fininshed my review. Please merge this. Thanks.

@youngtaekiim youngtaekiim merged commit 9f73794 into eclipse-pullpiri:main Sep 29, 2025
5 of 6 checks passed
@leeeunkoo leeeunkoo added the enhancement New feature or request label Sep 29, 2025
@leeeunkoo leeeunkoo added this to the Setting Dashboard milestone Sep 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[TASK] Change the name and operation of Container-related REST APIs on SettingsService for Dashboard

4 participants