Skip to content

RSDK-7359: remove deprecated robot_part_id from cloudMetadataResponse#830

Merged
allisonschiang merged 3 commits intoviamrobotics:mainfrom
allisonschiang:RSDK-7359
Mar 25, 2026
Merged

RSDK-7359: remove deprecated robot_part_id from cloudMetadataResponse#830
allisonschiang merged 3 commits intoviamrobotics:mainfrom
allisonschiang:RSDK-7359

Conversation

@allisonschiang
Copy link
Copy Markdown
Member

No description provided.

@github-actions github-actions bot added the safe to test committer is a member of this org label Mar 19, 2026
@allisonschiang allisonschiang added allow breaking Allow breaking changes by bypassing the buf-breaking check protos-compiled labels Mar 19, 2026
@allisonschiang allisonschiang requested a review from cheukt March 19, 2026 20:58
string location_id = 3;
string machine_id = 4;
string machine_part_id = 5;

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

do I need to make an announcement about this in breaking changes? since it was deprecated 2 years ago I'd hope it doesn't affect anything but I haven't gone through the deprecation process before

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

yep you should - in case anyone is still relying on this field

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

will approve after announcement made

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

do I need to give people time(one week?) before merging this?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ya, let's make that announcement and give them a week

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

done!

@allisonschiang allisonschiang added the ready-for-protos add this when you want protos to compile on every commit label Mar 25, 2026
@allisonschiang allisonschiang removed the ready-for-protos add this when you want protos to compile on every commit label Mar 25, 2026
@allisonschiang allisonschiang merged commit 2edca04 into viamrobotics:main Mar 25, 2026
12 of 16 checks passed
@allisonschiang allisonschiang deleted the RSDK-7359 branch March 25, 2026 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

allow breaking Allow breaking changes by bypassing the buf-breaking check protos-compiled safe to test committer is a member of this org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants