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

feat: sql query #721

Merged
merged 18 commits into from
Aug 8, 2024
Merged

feat: sql query #721

merged 18 commits into from
Aug 8, 2024

Conversation

boris-w
Copy link
Contributor

@boris-w boris-w commented Jul 15, 2024

Support structured json query base inside the data.

  • query base data endpoint
  • query builder component

@coveralls
Copy link
Collaborator

coveralls commented Jul 15, 2024

Pull Request Test Coverage Report for Build 10299837946

Details

  • 25 of 2566 (0.97%) changed or added relevant lines in 49 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.5%) to 17.046%

Changes Missing Coverage Covered Lines Changed/Added Lines %
apps/nextjs-app/src/features/app/blocks/setting/components/FormItem.tsx 0 1 0.0%
apps/nextjs-app/src/features/app/blocks/setting/query-builder/QueryBuilder.tsx 0 1 0.0%
apps/nextjs-app/src/features/app/components/setting/account/AddPassword.tsx 0 1 0.0%
apps/nextjs-app/src/features/auth/components/SignForm.tsx 0 1 0.0%
apps/nextjs-app/src/features/auth/pages/ForgetPasswordPage.tsx 0 1 0.0%
apps/nextjs-app/src/features/auth/pages/ResetPasswordPage.tsx 0 1 0.0%
apps/nextjs-app/src/pages/_app.tsx 0 1 0.0%
packages/core/src/auth/actions.ts 0 1 0.0%
packages/openapi/src/base/index.ts 0 1 0.0%
packages/sdk/src/components/base-query/index.ts 0 1 0.0%
Totals Coverage Status
Change from base Build 10299037390: -0.5%
Covered Lines: 14105
Relevant Lines: 82745

💛 - Coveralls

@boris-w boris-w force-pushed the feat/base-sql-query branch 3 times, most recently from 5b11130 to 128f82e Compare August 7, 2024 03:49
@boris-w boris-w marked this pull request as ready for review August 8, 2024 07:39
@boris-w boris-w requested review from caoxing9 and tea-artist August 8, 2024 07:39
async baseQuery(baseId: string, baseQuery: IBaseQuery): Promise<IBaseQueryVo> {
const { queryBuilder, fieldMap } = await this.parseBaseQuery(baseId, baseQuery, 0);
const query = queryBuilder.toQuery();
console.log('query', query);
Copy link
Contributor

Choose a reason for hiding this comment

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

necessary?

Copy link
Contributor

@caoxing9 caoxing9 left a comment

Choose a reason for hiding this comment

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

LGTM

@boris-w boris-w force-pushed the feat/base-sql-query branch from a4d9fa3 to 491c399 Compare August 8, 2024 09:48
@boris-w boris-w merged commit b457d2e into develop Aug 8, 2024
6 of 8 checks passed
@boris-w boris-w deleted the feat/base-sql-query branch August 8, 2024 10:09
AliceLanniste pushed a commit to AliceLanniste/teable that referenced this pull request Sep 1, 2024
* feat: sql query interface

* refactor: remove dbTableName from filter db provider

* feat: aggregation db-provider support group

* feat: simple base sql query

* refactor: refineExtendedFilterOperatorSchema

* refactor: base query

* chore: sql-query action rename query_data

* chore: query data action static

* fix: query filter and order aggregation field

* fix: default select all col includes aggregation col

* chore: move Error components to ui-lib

* chore: add chart com in ui-lib

* feat: add base-query components

* fix: typecheck

* feat: chart page

* fix: limit of select and orderby from aggregation and groupby

* feat: base query builder validation

* chore: base query log
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.

5 participants