-
Notifications
You must be signed in to change notification settings - Fork 44
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
Fixes issues #108 and #124. #125
base: master
Are you sure you want to change the base?
Conversation
`tox pep8,py27,py36` now passes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really nice work.
Thanks for your contribution.
@@ -32,6 +32,11 @@ class RTCClient(RTCBase): | |||
the url ends with 'jazz', otherwise to `False` if with 'ccm' | |||
(Refer to issue #68 for details) | |||
:type ends_with_jazz: bool | |||
:param attribute_map: (optional) Dictionary mapping RTC attributes to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually I don't get what this attribute_map
is for.
If you want to know all the alias, you can refer to field_alias
. See this link.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
attribute_map
does two things:
- It allows overriding the alias that is automatically created in
FieldBase.__initializeFromRaw
. - It is the mechanism for forcing the inclusion of an attribute with a "." in the name, which is the specific fix for issue Properties with "." in the name are not accessible #108
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello. Just following up. Is there anything else I can do to move this along?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can field_alias
in this link be used, instead of having a new attribute_map
?
tox pep8,py27,py36
now passes.