forked from pinterest/querybook
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Upstream test #6
Open
rohan-sh1
wants to merge
210
commits into
CAI-TECHNOLOGIES:pull/up-92d507d3f812eee1619edfb0cd10a7a3c282fddc
Choose a base branch
from
pinterest:master
base: pull/up-92d507d3f812eee1619edfb0cd10a7a3c282fddc
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Upstream test #6
rohan-sh1
wants to merge
210
commits into
CAI-TECHNOLOGIES:pull/up-92d507d3f812eee1619edfb0cd10a7a3c282fddc
from
pinterest:master
Conversation
This file contains 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
* chore: update yaml-loader * remove json-loader * fix storybook
The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-QS-3153490 Co-authored-by: snyk-bot <snyk-bot@snyk.io>
The following vulnerabilities are fixed by pinning transitive dependencies: - https://snyk.io/vuln/SNYK-PYTHON-SETUPTOOLS-3180412 Co-authored-by: snyk-bot <snyk-bot@snyk.io>
* chore: version upgrade a few packages * update snapshots * downgrade typescript * update qs ts deps
* chore: update richtext editor to hooks * update snapshot
* fix: json-bigint hasOwnProperty undefined issue * use cloneDeep
* fix: axios error display * update snapshots
* feat: add metastore config * comments * rename * remove session
Co-authored-by: Lily Li <lli6@expediagroup.com>
* add axios request timeout option * add option to include both notifyOnError and timeout
* feat: query engine search scroll * chore: refactor updateKeyword and SearchBar --------- Co-authored-by: Lily Li <lli6@expediagroup.com>
* feat: add search box to hide columns popup * feat: add clear button to hide columns search bar * chore: refactor filteredColumnNames and SearchBar
* Update searchbar css * self review * added constant to control when to show search bar
* feat: add user group db schema support * comments * disable user group indexing * addto sidebar * fix linter * more comments * fix linter * typo
* fix: sql-formmater * remove originalStatementText * try catch on whole query
* feat: support for general message in query validation plugin * addressed review comments
* feat: Cron schedule support * fix: PR comments
* Auth0 Integration * Subclass OktaLoginManager as Auth0LoginManager * Removed NoopAuth Unused code
* feat: add @mention support for the command input * address comments * fix table selector disappear issue * comments * remove id * use suggest * add comment
…1433) * fix: use match_phrase_prefix on full_name field for table suggestion * add gpt-4-turbo contextsize
* fix: revert to use searchConcise for mentioning a table * remove unused code * remove unused imports * comments
Previously, sending notifications to multiple email addresses (not users) at once didn't work.
* fix: add search parameters for table concise search * comment * disable eslint * fix linter
* fix querybook edit * fix: querybook edit access issue
* fix: move workflow link to source query tab * fix: move workflow link to source query tab * address comments * check link type * use util function for url check
* fix: move slack channel to top of table description * fix: move slack channel to top of table description * address comments * use pinnedCustomProperties to order properties * use pinnedCustomProperties to order properties
* chore: add openai new model gpt-4o * upgrade langchain
…#1450) * feat: add table links field * 3.34.0 * 3.34.0 * address comments * fix lint * fix lint
* fix: add back text2sql survey * fix linter
* add icon to table links * comments
* chore: add sampling context logs * comments * typo
* ui: Add user guide link for table sampling feature
* ui: Show sampled query execution as sampled * make metadata JSON column type
* ui: Add Query Nagging for Sampling
* feat: pass api_access_token to executor client * use execution metadata to store access token flag
* ui: disable tip for sampled queries
* chore: add parameters for the task of disable scheduled docs * fix linter
* fix: str_to_bool and sql-limiter * comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.