-
Notifications
You must be signed in to change notification settings - Fork 14
Closed
Labels
part:❓We need to figure out which part is affectedWe need to figure out which part is affectedpriority:❓We need to figure out how soon this should be addressedWe need to figure out how soon this should be addressedtype:enhancementNew feature or enhancement visitble to usersNew feature or enhancement visitble to users
Milestone
Description
What's needed?
The way this field is named in the Component message is misleading:
frequenz-api-common/proto/frequenz/api/common/v1/microgrid/components/components.proto
Lines 134 to 135 in 5265439
| // The metadata specific to the component category type. | |
| ComponentCategoryMetadataVariant category_type = 5; |
The field name was left as is since older times, and it probably needs to be updated to reflect its latest type.
Proposed solution
We could just rename it to component_metadata.
Use cases
No response
Alternatives and workarounds
No response
Additional context
No response
Metadata
Metadata
Assignees
Labels
part:❓We need to figure out which part is affectedWe need to figure out which part is affectedpriority:❓We need to figure out how soon this should be addressedWe need to figure out how soon this should be addressedtype:enhancementNew feature or enhancement visitble to usersNew feature or enhancement visitble to users