Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/get properties #64

Merged
merged 2 commits into from
Jan 20, 2025

Conversation

lucaato
Copy link
Contributor

@lucaato lucaato commented Dec 5, 2024

No description provided.

@lucaato lucaato force-pushed the feature/get-properties branch 3 times, most recently from e001276 to ecd9a6c Compare December 5, 2024 16:55
@lucaato lucaato marked this pull request as ready for review December 5, 2024 17:28
proto/astarteplatform/msghub/property.proto Outdated Show resolved Hide resolved
proto/astarteplatform/msghub/property.proto Outdated Show resolved Hide resolved
proto/astarteplatform/msghub/property.proto Outdated Show resolved Hide resolved
@lucaato lucaato force-pushed the feature/get-properties branch 2 times, most recently from 2a13e9e to 03a4f66 Compare December 6, 2024 11:18
@lucaato lucaato requested a review from harlem88 December 6, 2024 11:18
@lucaato lucaato force-pushed the feature/get-properties branch from 03a4f66 to 9d70cee Compare December 6, 2024 11:41
@lucaato lucaato requested a review from rgallor December 6, 2024 13:28
@harlem88 harlem88 requested review from sorru94 and joshuachp December 6, 2024 13:39
@lucaato lucaato force-pushed the feature/get-properties branch from 9d70cee to 88f8c9e Compare December 9, 2024 11:09
@lucaato lucaato requested a review from joshuachp December 9, 2024 11:52
proto/astarteplatform/msghub/property.proto Outdated Show resolved Hide resolved
@rgallor
Copy link
Collaborator

rgallor commented Dec 9, 2024

Since the sdk defines the major version as i32, would we like to change uint32 into int32 here?

@lucaato lucaato force-pushed the feature/get-properties branch 4 times, most recently from 7260140 to 4323dfb Compare December 10, 2024 16:11
Copy link
Collaborator

@harlem88 harlem88 left a comment

Choose a reason for hiding this comment

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

LGTM. Add a few lines to the changelog about these changes.

@lucaato lucaato force-pushed the feature/get-properties branch from 4323dfb to 4baf25c Compare December 13, 2024 11:59
lucaato and others added 2 commits December 13, 2024 13:00
Co-authored-by: Riccardo Gallo <riccardo.gallo@secomind.com>
Signed-off-by: Luca Arato <luca.arato@secomind.com>
Co-authored-by: Luca Arato <luca.arato@secomind.com>
Signed-off-by: Luca Arato <luca.arato@secomind.com>
@lucaato lucaato force-pushed the feature/get-properties branch from 4baf25c to c7d3fb3 Compare December 13, 2024 12:01
@lucaato lucaato requested a review from harlem88 December 13, 2024 13:26
@harlem88 harlem88 merged commit 370be08 into astarte-platform:master Jan 20, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants