search

frequenz-floss / frequenz-api-reporting

Frequenz Reporting API gRPC/protobuf specifications
MIT License
1 stars 6 forks source link

Remove `states` parameter from `IncludeOptions` #48

Closed stefan-brus-frequenz closed 3 weeks ago

stefan-brus-frequenz commented 2 months ago

What's needed?

The reporting API should not be responsible for exposing component states.

Proposed solution

The states field of IncludeOptions should be removed.

Use cases

No response

Alternatives and workarounds

No response

Additional context

No response

cwasicki commented 1 month ago

The requirements have changed and we have to keep this option. At least for now, whether we can remove it at all needs to be decided at a later stage.

stefan-brus-frequenz commented 3 weeks ago

Closing this