Skip to content

SDK reference fixes for SCAL-251659, SCAL-297673, and SCAL-337086 - #562

Open
ShashiSubramanya wants to merge 3 commits into
mainfrom
sdk-reference-doc-format
Open

ShashiSubramanya wants to merge 3 commits into
mainfrom
sdk-reference-doc-format

Conversation

@ShashiSubramanya

Copy link
Copy Markdown
Collaborator

No description provided.

@vercel

vercel Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
developer-docs Ready Ready Preview Oct 16, 2026 2:46pm UTC

Request Review

Comment thread scripts/Converter/index.ts Fixed
@@ -0,0 +1,212 @@
import React, { useEffect, useRef, useState } from 'react';

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is this new ui
can you share where this is present in the ui

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is for the filter component in the Actions page (SCAL-251659 fix). I have added this to allow filtering action IDs by embed type, because the actions list is hard to use now that there are many actions and embed types.

You can view the filter component on this page:
https://developer-docs-git-sdk-reference-doc-format-thoughtspot-site.vercel.app/docs/Enumeration_Action#_index

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add ui like similar to the admin tab

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ShashiSubramanya Can you please check this above comment rest all code

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mean the admin tab in the product UI?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like this
image

@shivam-kumar-ts shivam-kumar-ts left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Just check this in follow up pr

Image

This branch was successfully deployed

1 active deployment
Preview — 96edae63 Deployed Oct 16, 2026 by vercel[bot]
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