Skip to content

Conversation

@denislavstanchev
Copy link
Contributor

No description provided.

…e from JavaScript to TypeScript and enhance type definitions
@roomote
Copy link

roomote bot commented Nov 18, 2025

Rooviewer Clock   See task on Roo Cloud

Review complete. Latest changes rename test files to include _test suffix and refactor the addProfileImage test to use extracted constants and more precise assertions. No new issues found.

  • Update SmartsheetClient interface to use WebhooksApi type instead of any for the webhooks property
  • Export webhook types from index.ts for consistency with other modules
Previous reviews

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

ggoranov-smar
ggoranov-smar previously approved these changes Nov 18, 2025
ggoranov-smar
ggoranov-smar previously approved these changes Nov 18, 2025
@denislavstanchev denislavstanchev merged commit d5c9e9e into mainline Nov 18, 2025
15 of 21 checks passed
@denislavstanchev denislavstanchev deleted the webhooks-add-typing-and-tests branch November 18, 2025 12:33
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.

3 participants