search

astarte-platform / astarte-device-sdk-esp32

Astarte device SDK for ESP32 devices, based on esp-idf
Apache License 2.0
13 stars 11 forks source link

Avoid duplicates in introspection #192

Closed sorru94 closed 1 year ago

sorru94 commented 1 year ago

Still open to discussion:

Close #191

harlem88 commented 1 year ago

Still open to discussion:

  • Should we notify the user through an error when he attempts to install the same interface twice?

Why should we generate an error ? I think we should only generate an error if he attempts to install the same interface with < major, because installing an interface with a > major is a valid operation.

sorru94 commented 1 year ago

Still open to discussion:

  • Should we notify the user through an error when he attempts to install the same interface twice?

Why should we generate an error ? I think we should only generate an error if he attempts to install the same interface with < major, because installing an interface with a > major is a valid operation.

As discussed externally, we will return an error only when attempting to replace an interface with an incompatible type or ownership or when attempting to install an interface with a smaller version.