-
Notifications
You must be signed in to change notification settings - Fork 413
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
add query-tagging attribute to connection #1202
Conversation
4c679cb
to
e782c29
Compare
Validated this locally using one hyper datasource (cannot change the value for queryTagging) and one postgres connection (attribute successfullyl updated) |
@@ -52,6 +69,7 @@ def from_response(cls, resp, ns) -> List["ConnectionItem"]: | |||
connection_item.server_address = connection_xml.get("serverAddress", None) | |||
connection_item.server_port = connection_xml.get("serverPort", None) | |||
connection_item.username = connection_xml.get("userName", None) | |||
connection_item._query_tagging = connection_xml.get("queryTaggingEnabled", None) |
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.
Just double checking we don't need to use string_to_bool()
here like we did on connection_item.embed_password
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.
good catch. Added that and tests to cover it.
* Implement custom view objects (#1195) * Fix bug in update-datasources before 3.15 (#1203) (fixes #1072) * catch exceptions from ServerInfo (#1204) * add query-tagging attribute to connection (#1202) (add explanation for why it doesn't work on hyper) --------- Co-authored-by: Marwan Baghdad <mrwanbaghdad76@gmail.com> Co-authored-by: jorwoods <jorwoods@users.noreply.github.com> Co-authored-by: Brian Cantoni <bcantoni@salesforce.com> Co-authored-by: TrimPeachu <77048868+TrimPeachu@users.noreply.github.com> Co-authored-by: Stu Tomlinson <stu@nosnilmot.com> Co-authored-by: Jeremy Harris <jercharris89@gmail.com>
need to figure out how to test this before merging