Skip to content

Specify list results in SDK methods#116

Merged
albertokarakoutev-smar merged 2 commits intomainlinefrom
bugfix/specify-list-result-responses
Dec 12, 2025
Merged

Specify list results in SDK methods#116
albertokarakoutev-smar merged 2 commits intomainlinefrom
bugfix/specify-list-result-responses

Conversation

@albertokarakoutev-smar
Copy link
Contributor

Pull Request

Description

Some return types of the SDK methods and subsequently the public API support both a singleton item result and a list of them. This MR adds the proper notation for those methods using Union types.

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Code refactoring
  • Other (please describe):

Environment Information

  • Smartsheet API Version: 2.0
  • Smartsheet Python SDK Version: 3.7.1
  • Python Version: 3.7

What Changes Were Made

Update the return types of the affected methods.

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have updated the relevant files in docs-source/ (see Documentation guidelines)
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@roomote
Copy link
Contributor

roomote bot commented Dec 11, 2025

Rooviewer Clock   Follow along on Roo Cloud

Reviewing your PR now. I'll share feedback in a few minutes!

@albertokarakoutev-smar albertokarakoutev-smar merged commit cc52b1a into mainline Dec 12, 2025
10 checks passed
@albertokarakoutev-smar albertokarakoutev-smar deleted the bugfix/specify-list-result-responses branch December 12, 2025 12:13
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