From 4327c97b3a1dbb572428b82d9b748e635135ff4f Mon Sep 17 00:00:00 2001 From: David Thole Date: Thu, 3 Oct 2024 14:49:15 -0500 Subject: [PATCH 1/2] Fixing a typo in the available keys, forgetting the comma. Also, to keep documentation consistent, just string joined the valid keys so it doesn't need to be in two spots --- QualtricsAPI/Survey/responses.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/QualtricsAPI/Survey/responses.py b/QualtricsAPI/Survey/responses.py index 57ee02f..acf5372 100644 --- a/QualtricsAPI/Survey/responses.py +++ b/QualtricsAPI/Survey/responses.py @@ -220,7 +220,7 @@ def get_survey_responses(self, survey=None, verify=None, **kwargs): 'startDate', 'endDate', 'timeZone', - 'breakoutSets' + 'breakoutSets', 'sortByLastModifiedDate', 'filterId', 'embeddedDataIds', @@ -229,7 +229,7 @@ def get_survey_responses(self, survey=None, verify=None, **kwargs): ] for key in list(kwargs.keys()): - assert key in valid_keys, "Hey there! You can only pass in parameters with names in the list, ['useLabels', 'includeLabelColumns', 'exportResponsesInProgress', 'limit', 'seenUnansweredRecode', 'multiselectSeenUnansweredRecode', 'includeDisplayOrder', 'startDate', 'endDate', 'timeZone']" + assert key in valid_keys, f"Hey there! You can only pass in parameters with names in the list, [{', ', valid_keys}]" if key == 'useLabels': assert 'includeLabelColumns' not in list(kwargs.keys( )), 'Hey there, you cannot pass both the "includeLabelColumns" and the "useLabels" parameters at the same time. Please pass just one and try again.' From cbc4fbe7a98147f9d459313c8e648d06cac44354 Mon Sep 17 00:00:00 2001 From: David Thole Date: Thu, 3 Oct 2024 14:55:49 -0500 Subject: [PATCH 2/2] Fixing a typo in my own code, in the join statement --- QualtricsAPI/Survey/responses.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/QualtricsAPI/Survey/responses.py b/QualtricsAPI/Survey/responses.py index acf5372..38f8770 100644 --- a/QualtricsAPI/Survey/responses.py +++ b/QualtricsAPI/Survey/responses.py @@ -229,7 +229,7 @@ def get_survey_responses(self, survey=None, verify=None, **kwargs): ] for key in list(kwargs.keys()): - assert key in valid_keys, f"Hey there! You can only pass in parameters with names in the list, [{', ', valid_keys}]" + assert key in valid_keys, f"Hey there! You can only pass in parameters with names in the list, [{', '.join(valid_keys)}]" if key == 'useLabels': assert 'includeLabelColumns' not in list(kwargs.keys( )), 'Hey there, you cannot pass both the "includeLabelColumns" and the "useLabels" parameters at the same time. Please pass just one and try again.'