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

Smartsheet connection load bot ability to get single row #3495

Closed
TCMMatthiasFankhauser opened this issue Nov 21, 2023 · 2 comments
Closed
Assignees

Comments

@TCMMatthiasFankhauser
Copy link

TCMMatthiasFankhauser commented Nov 21, 2023

The current implementation of the smartsheet connection load bot is simply returning the entire sheet.

For our usecase with Amazon ATS, we hould be able to query for a single row in the smartsheet. This would allow us to more easily load all corresponding salesforce data & sync without the risk of interfering with unrelated rows just because they are also loaded.

The smartsheet docs ( https://smartsheet.redoc.ly/#section/API-Basics/Formatting ) lists a GET command that looks like its able to handle a rowid param ( *GET sheets/{sheetId}/rows/{rowId} *)

But i could not get it to work on the fly. Is it possible to get a variation on the smartsheet connector bot that would work for a single row only?

Currently our workaround is to load all rows in the Sheet and discarding all but one. Not ideal from a scalability point of view. And as mentioned, having more than one smartsheet row loaded in this case can contribute to faulty syncs.

┆Issue is synchronized with this Clickup task by Unito

@zachelrath zachelrath self-assigned this Nov 21, 2023
@zachelrath
Copy link
Contributor

I added this feature here in this PR: ues-io/integrations#32

@TCMMatthiasFankhauser
Copy link
Author

@zachelrath tested it with the help of @abelcloudmasters . Works well

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

No branches or pull requests

2 participants