Config: remove version id from response body#822
Conversation
📝 WalkthroughWalkthroughThe PR removes the explicit ChangesConfigVersion Model Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
c8fa154 to
f870f61
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
|
||
|
|
||
| class ConfigVersionPublic(ConfigVersionBase): | ||
| id: UUID = Field(description="Unique id for the configuration version") |
There was a problem hiding this comment.
Just confirm, was this removed only because the config version did not require this API? Also, if it has been removed from here, then corresponding changes may also be required on the frontend side? otherwise, the frontend might crash.
Summary
Target issue is #624
Notes