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

fix: relax twin collection version type constraint #729

Merged
merged 3 commits into from
Jan 6, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions azext_iot/assets/user_messages.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,14 +26,14 @@ def error_param_top_out_of_bounds(upper_limit=None):

def info_param_properties_device(include_mqtt=True, include_http=False):
http_content = (
"For http messaging - application properties are sent using iothub-app-<name>=value, for instance "
"iothub-app-myprop=myvalue. System properties are generally prefixed with iothub-<name> like iothub-correlationid "
"For http messaging - application properties are sent using iothub-app-{name}=value, for instance "
"iothub-app-myprop=myvalue. System properties are generally prefixed with iothub-{name} like iothub-correlationid "
"but there are exceptions such as content-type and content-encoding. "
)

mqtt_content = (
"For mqtt messaging - you are able to send system properties using "
"$.<name>=value. For instance $.cid=12345 sets the system correlation Id property. "
"$.{name}=value. For instance $.cid=12345 sets the system correlation Id property. "
"Other system property identifier examples include $.ct for content type, "
"$.mid for message Id and $.ce for content encoding. "
)
Expand Down
2 changes: 1 addition & 1 deletion azext_iot/deviceupdate/params.py
Original file line number Diff line number Diff line change
Expand Up @@ -343,7 +343,7 @@ def load_deviceupdate_arguments(self, _):
"deployment_id",
options_list=["--deployment-id", "--did"],
help="The caller-provided deployment Id. This cannot be longer than 73 characters, "
"must be all lower-case, and cannot contain '&', '^', '[', ']', '{', '}', '|', '<', '>', "
"must be all lower-case, and cannot contain `&`, `^`, `[`, `]`, `{`, `}`, `|`, `<`, `>`, "
"forward slash, backslash, or double quote.",
)
context.argument(
Expand Down
2 changes: 1 addition & 1 deletion azext_iot/sdk/dps/service/models/twin_collection_py3.py
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ class TwinCollection(Model):

_attribute_map = {
'additional_properties': {'key': '', 'type': '{object}'},
'version': {'key': 'version', 'type': 'long'},
'version': {'key': 'version', 'type': 'object'}, # TODO 'long' -> 'object': rest API spec is not accurate. More than int is supported.
'count': {'key': 'count', 'type': 'int'},
'metadata': {'key': 'metadata', 'type': 'Metadata'},
}
Expand Down
Loading