fix: ensure interface-provided 'orderBy' parameter overrides view's default sort #463
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
tea-artist
reviewed
Mar 18, 2024
boris-w
force-pushed
the
fix/view-orderby
branch
2 times, most recently
from
5135602
to
ed5837a
Compare
boris-w
changed the title
boris-w
force-pushed
the
fix/view-orderby
branch
from
58fd704
to
2b64fc2
Compare
tea-artist
approved these changes
Mar 18, 2024
tea-artist
added a commit
that referenced
this pull request
Mar 20, 2024
* feat: support excel import (#431) * feat: support `csv` import * perf: optimise import memory expend * fix: `openapi` upload type error * test: compatible ci attachment download * feat: add import file icon * feat: support `excel` import feat: limit excel file size refactor: rewrite import font-end * chore: add `sheetjs` license * feat: update import url validate strategy * test: add excel import e2e * feat: add logger in import records * fix: type and lint error * chore: replace axios to node-fetch in nestjs * test: update import-table e2e * test: update import-table delay time * chore: delete `sheetjs` license * feat: add analyze loading * chore: export upload-panel component * feat: integrate github login (#432) * feat: integrate gitHub login * feat: dynamic social auth module * feat: add github auth env validation * chore: remove electron (#438) * feat: credit check (#436) * feat: credit check * test: exceed credit limit * refactor: more appropriate email template structure (#437) * chore: make pnpm automatically install the specified version of Node.js (#433) * chore: remove nvm step * fix: the error of copying and pasting empty content (#443) * fix: the error of copying and pasting empty content * ci: fixed node version * ci: pnpm newline handling * feat: compatible `tsv` import (#446) * fix: rename link results in illegal data (#448) * fix: unexpected horizontal lines on login page (#447) * test: import test file save in tmp dir (#449) * docs: add deploy on Zeabur in Readme (#451) * docs: add deploy on Zeabur in Readme * Update README.md * Update README.md * build: image support `multi-platforms` & app image support `csp-open` (#445) * build: app image support `csp-open` * feat: docker image multi-platforms support * fix: allow default db address (#461) * feat: forgot password and setup new password (#452) * feat: forgot password and setup new password * chore: remove useless code * fix: use instance test context creator * fix: type error * fix: sdk build config for *.spec.tsx * fix: execute repeat when import upload (#464) * fix: notify jump url & upgrade deps (#465) * fix: notify jump url & upgrade deps * feat: update `PUBLIC_DATABASE_PROXY` env * fix: ensure interface-provided 'orderBy' parameter overrides view's default sort (#463) * fix: ensure interface-provided 'orderBy' parameter overrides view's default sort * fix: existence of identical fields sorted, with the later overwriting the earlier * chore: signin error alerts * feat: support google oatuh2 (#466) * refactor: api for adjusting the order (#453) * fix: i18n * docs: star distribution * refactor: view order * refactor: view+table+base updateOrder * feat: ci add `coverallsapp/github-action` (#468) * feat: ci add `coverallsapp/github-action` * fix: vitest `coverage.reporter` config (#469) * fix: vitest `coverage.reporter` config * chore(example): update dockers examples (#474) * docs: Update header titles for consistency (#455) * docs: add one-click deploy on sealos. (#462) Signed-off-by: zzjin <tczzjin@gmail.com> * chore: Update constants.ts (#457) minor fix * refactor: row order (#473) * refactor: row order * fix: sqlite test --------- Signed-off-by: zzjin <tczzjin@gmail.com> Co-authored-by: Mike <caoxing9@gmail.com> Co-authored-by: Boris <boris2code@outlook.com> Co-authored-by: Anjorin Damilare <damilareanjorin1@gmail.com> Co-authored-by: Yuhang <2312744987@qq.com> Co-authored-by: Pengap <penganpingprivte@gmail.com> Co-authored-by: emmanuel <154705254+codesmith-emmy@users.noreply.github.com> Co-authored-by: zzjin <zzjin@users.noreply.github.com> Co-authored-by: Ikko Eltociear Ashimine <eltociear@gmail.com>
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.