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

Remove ExplicitTimestamp from server owned interface #453

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

eddbbt
Copy link
Contributor

@eddbbt eddbbt commented Jul 15, 2024

Explicit timestamp is added and removed from modals and interface, depending on the selected owner, and checked at settings change.
Does not enforce explicit_timestamp for already existing interface or after json manual editing

Partially resolves #447

@eddbbt eddbbt force-pushed the server-interface-timestamp branch 2 times, most recently from c80ff03 to ad02c51 Compare July 15, 2024 16:27
@eddbbt eddbbt marked this pull request as ready for review July 15, 2024 16:30
@eddbbt eddbbt force-pushed the server-interface-timestamp branch 3 times, most recently from d2daa67 to 883550b Compare July 16, 2024 15:57
@eddbbt eddbbt force-pushed the server-interface-timestamp branch 2 times, most recently from 9d2f959 to 68e4c6b Compare July 17, 2024 14:27
@eddbbt eddbbt force-pushed the server-interface-timestamp branch from 68e4c6b to b9deb96 Compare July 18, 2024 07:57
@eddbbt eddbbt requested a review from Pavinati July 18, 2024 07:57
@eddbbt eddbbt force-pushed the server-interface-timestamp branch from b9deb96 to 3fcc34b Compare July 18, 2024 09:34
Explicit timestamp is added and removed from modals and interface, depending on the selected owner, and checked at settings change

Signed-off-by: Eddy Babetto <[email protected]>
@eddbbt eddbbt force-pushed the server-interface-timestamp branch from 3fcc34b to 9f732a2 Compare July 18, 2024 13:00
@eddbbt eddbbt requested a review from Pavinati July 18, 2024 13:02
@Pavinati Pavinati merged commit 0478bd4 into astarte-platform:master Jul 18, 2024
4 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.

Payload for server owned interfaces: explicit timestamp not allowed
2 participants