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

Update insights endpoints to clarify optional/required params [INS-177] #3023

Merged
merged 3 commits into from
Nov 25, 2024

Conversation

adam-saucelabs
Copy link
Contributor

adding note to two endpoints that did not have information about XOR on start/end or time_range

Description

Motivation and Context

Types of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation fix (typos, incorrect content, missing content, etc.)

adding note to two endpoints that did not have information about XOR on start/end or time_range
Copy link

Deploy preview ready for 3023!
https://docs.dev.saucelabs.net/pr-preview/pr-3023

updating copy for start/end/time_range and parameter sorting for 

- /v1/analytics/tests                 
- /v1/analytics/insights/test-metrics 
- /v1/analytics/trends/tests          
- /v1/analytics/trends/builds_tests
Copy link

Deploy preview ready for 3023!
https://docs.dev.saucelabs.net/pr-preview/pr-3023

Copy link

Deploy preview ready for 3023!
https://docs.dev.saucelabs.net/pr-preview/pr-3023

@pawel-swiecki-saucelabs

lgtm :)

@adam-saucelabs
Copy link
Contributor Author

@pawel-swiecki-saucelabs approved on slack

@adam-saucelabs adam-saucelabs merged commit 66914c0 into main Nov 25, 2024
3 checks passed
@adam-saucelabs adam-saucelabs deleted the update-insights-optional-params-INS-177 branch November 25, 2024 16:27
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.

2 participants