Use Mixpanel instead of Cloudflare analytics #19
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
🎉 Snyk checks have passed. No issues have been found so far.✅ security/snyk check is complete. No issues have been found. (View Details) ✅ license/snyk check is complete. No issues have been found. (View Details) |
ashubham
changed the title
shikharbsar
approved these changes
May 22, 2025
shikharbsar
added a commit
that referenced
this pull request
Jul 1, 2025
* encode state in /callback * Fix headers / liveboard url * Use proxy for get TML + refactor * Proxy the data request to avoid 403 * Handle url sanitization (#3) * HAndle url sanitization * Remove https only restriction * Add code for calling /v1/v2/auth/token/authorize once implemented * Expose Data sources as resources (#4) * Update readme * Handle 0 answers case for permission * increase num queries * Split tools into multiple (#5) * add error handling * Add API server routes (#6) * Add API server routes * newline imports * Remove relevant data + error handling * fix tests (#8) * error on no relevant qs * update readme * udapte readme * Increase auth time (#10) * Introduce validity time for token fetched from TS * add token ttl * Update time to 60 secs * Increase validity time for TS token to 30 days * Remove proxy for export-answer-report * update description * Update README.md * Make base64 encoding url safe (#14) * Add the MCP server logo and TS logo (#15) * Lint + Stdio (#17) * Lint + Stdio * update coverage include * Usage analytics (#18) * Update approval page (#20) * Update the UI for approval dialog * remove unnecessary validation * Use Mixpanel instead of Cloudflare analytics (#19) * Use Mixpanel instead of Cloudflare anaylitics * fix tests * update relevant question desc * update relevant question desc * Add checkbox for TS legal terms of service (#24) * Fix Mixpanel (#26) * sample_rate: 1 * index.html (#27) * update readme * update readme * Format LICENSE * Update Readme * update readme * update readme * workaround for SCAL-258056 * Update deps * 0.5.0 * Support Bearer token MCP (#28) * Add locale header for every request with ThoughtSpot Rest client (#31) * Add locale header for every request with ThoughtSpot Rest client * clear logs * do not add Accept-Language header if already present in context * fix lint errors * fix lint error 2 * add demo video * Add Video * Fix readme * Fix readme * Update README.md * add loom link * Add tests for oauth flow and servers (#33) * add tests for oauth flow and servers * lint fixes --------- Co-authored-by: Ashish Shubham <ashish.shubham@thoughtspot.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.