tongchong
  • Joined on Dec 09, 2022
Loading Heatmap…

tongchong synced commits to slurm_adapter at tongchong/SCOW from mirror

5 hours ago

tongchong synced commits to file-transfer at tongchong/SCOW from mirror

  • 29cdf08719 skip了queryTransferFiles.test.ts的测试

5 hours ago

tongchong synced commits to master at tongchong/phaser from mirror

6 hours ago

tongchong synced commits to main at tongchong/query from mirror

  • 2bab18520d chore: adding the nx-cloud task-runner (#5143) * adding the nx-cloud task-runner * empty commit to trigger ci with warm cache * retrying after adding workspace to TanStack organization * another empty commit... * another empty commit... * this is why I should never use the in-browser conflict resolver... * another empty commit... * another empty commit... * Update packages/query-core/src/tests/hydration.test.tsx Co-authored-by: Dominik Dorfmeister <office@dorfmeister.cc> * rm console.log left in query core * empty commit to trigger re-build * updating nx cloud dependency * empty commit to attempt to recreate unexpected cache miss * gather hash data to check * empty commit * invalidating cache again * removing old key and adding read-only key to source control * empty commit * updating build task * adding rollup to cacheable operations * fixing the project json file with better globs * removing test:format and test:build as cacheable operations * updating package scripts for better terminal logging * removing comment left in * bumping to latest cloud runner version * updating pnpm-lock.yaml file * adding skip-nx-cache to publish script --------- Co-authored-by: Dominik Dorfmeister <office@dorfmeister.cc>
  • 9e92e5ecd3 docs: Update useQueries.md (#5180) * Update useQueries.md Specifying that the return value is ordered, to help users understand how to match up the results with their inputs: https://stackoverflow.com/questions/72045306/usequeries-how-to-get-response-for-a-certain-query * Elaborated on how the order of the useQueries return value matches the input order
  • 9759fee0b8 docs: Update paginated-queries.md (#5191)
  • Compare 3 commits »

6 hours ago

tongchong synced commits to feature/queryOptions at tongchong/query from mirror

  • 2830d11be8 Merge branch 'alpha' into feature/queryOptions
  • eff7f03c44 Merge branch 'main' into alpha # Conflicts: # package.json # pnpm-lock.yaml
  • 2bab18520d chore: adding the nx-cloud task-runner (#5143) * adding the nx-cloud task-runner * empty commit to trigger ci with warm cache * retrying after adding workspace to TanStack organization * another empty commit... * another empty commit... * this is why I should never use the in-browser conflict resolver... * another empty commit... * another empty commit... * Update packages/query-core/src/tests/hydration.test.tsx Co-authored-by: Dominik Dorfmeister <office@dorfmeister.cc> * rm console.log left in query core * empty commit to trigger re-build * updating nx cloud dependency * empty commit to attempt to recreate unexpected cache miss * gather hash data to check * empty commit * invalidating cache again * removing old key and adding read-only key to source control * empty commit * updating build task * adding rollup to cacheable operations * fixing the project json file with better globs * removing test:format and test:build as cacheable operations * updating package scripts for better terminal logging * removing comment left in * bumping to latest cloud runner version * updating pnpm-lock.yaml file * adding skip-nx-cache to publish script --------- Co-authored-by: Dominik Dorfmeister <office@dorfmeister.cc>
  • 039bcc853d release: v5.0.0-alpha.11
  • 9e92e5ecd3 docs: Update useQueries.md (#5180) * Update useQueries.md Specifying that the return value is ordered, to help users understand how to match up the results with their inputs: https://stackoverflow.com/questions/72045306/usequeries-how-to-get-response-for-a-certain-query * Elaborated on how the order of the useQueries return value matches the input order
  • Compare 16 commits »

6 hours ago

tongchong synced commits to alpha at tongchong/query from mirror

  • eff7f03c44 Merge branch 'main' into alpha # Conflicts: # package.json # pnpm-lock.yaml
  • 2bab18520d chore: adding the nx-cloud task-runner (#5143) * adding the nx-cloud task-runner * empty commit to trigger ci with warm cache * retrying after adding workspace to TanStack organization * another empty commit... * another empty commit... * this is why I should never use the in-browser conflict resolver... * another empty commit... * another empty commit... * Update packages/query-core/src/tests/hydration.test.tsx Co-authored-by: Dominik Dorfmeister <office@dorfmeister.cc> * rm console.log left in query core * empty commit to trigger re-build * updating nx cloud dependency * empty commit to attempt to recreate unexpected cache miss * gather hash data to check * empty commit * invalidating cache again * removing old key and adding read-only key to source control * empty commit * updating build task * adding rollup to cacheable operations * fixing the project json file with better globs * removing test:format and test:build as cacheable operations * updating package scripts for better terminal logging * removing comment left in * bumping to latest cloud runner version * updating pnpm-lock.yaml file * adding skip-nx-cache to publish script --------- Co-authored-by: Dominik Dorfmeister <office@dorfmeister.cc>
  • 039bcc853d release: v5.0.0-alpha.11
  • 9e92e5ecd3 docs: Update useQueries.md (#5180) * Update useQueries.md Specifying that the return value is ordered, to help users understand how to match up the results with their inputs: https://stackoverflow.com/questions/72045306/usequeries-how-to-get-response-for-a-certain-query * Elaborated on how the order of the useQueries return value matches the input order
  • 9759fee0b8 docs: Update paginated-queries.md (#5191)
  • Compare 8 commits »

6 hours ago

tongchong synced commits to release at tongchong/appsmith from mirror

6 hours ago

tongchong synced commits to mobile/v1/qa at tongchong/appsmith from mirror

6 hours ago

tongchong synced commits to mobile/v1/perf at tongchong/appsmith from mirror

6 hours ago

tongchong synced commits to lightning-demo/31Mar/DS-pod at tongchong/appsmith from mirror

  • 3b8d12d1ea Fix issue with import
  • 583c122460 Merge branch 'feat/deduplicate-datasources-pages' into lightning-demo/31Mar/DS-pod
  • 4e2c266f3e Merge branch 'feat/deduplicate-button-and-icon' into feat/deduplicate-datasources-pages
  • dd3c5b128b Merge branch 'feat/deduplicate-button-and-icon' into feat/deduplicate-datasources-pages
  • e99adc4235 Back button becomes a link
  • Compare 17 commits »

6 hours ago

tongchong synced commits to fix/git-checkout-404 at tongchong/appsmith from mirror

6 hours ago

tongchong synced commits to fix/border-fix at tongchong/appsmith from mirror

  • 2280385842 Merge branch 'release' of github.com:appsmithorg/appsmith into fix/border-fix
  • 778922bd74 chore: Updating tests which will fail on app invites getting merged (#21949) ## Description > Updating tests which will fail when app invites are merged on EE. Along with that, updating permission check for the invite modal content on app level. Fixes [#21952](https://github.com/appsmithorg/appsmith/issues/21952) ## Type of change - Chore (housekeeping or task changes that don't impact user perception) ## How Has This Been Tested? > Ran the cypress tests on local and it works fine. - Manual - Cypress ## Checklist: ### Dev activity - [x] My code follows the style guidelines of this project - [x] I have performed a self-review of my own code - [x] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [x] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag ### QA activity: - [ ] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test
  • dffbf3d97c ci: Adding Identified Flaky tests app link into Git comments section in a PR (#21966) ## Description - This PR adds the Flaky list application link into the git comments section on completion of the run, so the PR owners have easy access to it. ## Type of change - Yml file update ## Checklist: ### QA activity: - [X] Added Test Plan Approved label after reviewing all changes
  • 63f63365d9 fix: Omnibar not showing recent entities (#21784) ## Description The Omnibar is supposed to update its order to ensure recent items are listed first. This stopped working at some time and this PR fixes it > Omnibar will list recently accessed items first Fixes #18963 ## Type of change - Bug fix (non-breaking change which fixes an issue) ## How Has This Been Tested? - Manual - Cypress ### Test Plan > Add Testsmith test cases links that relate to this PR ### Issues raised during DP testing > Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR) ## Checklist: ### Dev activity - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag ### QA activity: - [ ] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test
  • bb23521a51 fix: Entity Explorer test case (#21939) ## Description Fix test case failing because of changes in https://github.com/appsmithorg/appsmith/pull/21789 It caused failures in tests that had a move of action involved. This change will update the test cases to open the entity collapsible section after a move of action > Fix failing test case ## Type of change - Bug fix (non-breaking change which fixes an issue) ## How Has This Been Tested? - Cypress ### Test Plan > Add Testsmith test cases links that relate to this PR ### Issues raised during DP testing > Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR) ## Checklist: ### Dev activity - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag ### QA activity: - [ ] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test
  • Compare 6 commits »

6 hours ago

tongchong synced commits to feat/enable-fork at tongchong/appsmith from mirror

  • 6cb5a98f10 chore: fix cypress import
  • 43e747426f Merge branch 'release' of github.com:appsmithorg/appsmith into feat/enable-fork
  • 778922bd74 chore: Updating tests which will fail on app invites getting merged (#21949) ## Description > Updating tests which will fail when app invites are merged on EE. Along with that, updating permission check for the invite modal content on app level. Fixes [#21952](https://github.com/appsmithorg/appsmith/issues/21952) ## Type of change - Chore (housekeeping or task changes that don't impact user perception) ## How Has This Been Tested? > Ran the cypress tests on local and it works fine. - Manual - Cypress ## Checklist: ### Dev activity - [x] My code follows the style guidelines of this project - [x] I have performed a self-review of my own code - [x] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [x] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag ### QA activity: - [ ] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test
  • dffbf3d97c ci: Adding Identified Flaky tests app link into Git comments section in a PR (#21966) ## Description - This PR adds the Flaky list application link into the git comments section on completion of the run, so the PR owners have easy access to it. ## Type of change - Yml file update ## Checklist: ### QA activity: - [X] Added Test Plan Approved label after reviewing all changes
  • 63f63365d9 fix: Omnibar not showing recent entities (#21784) ## Description The Omnibar is supposed to update its order to ensure recent items are listed first. This stopped working at some time and this PR fixes it > Omnibar will list recently accessed items first Fixes #18963 ## Type of change - Bug fix (non-breaking change which fixes an issue) ## How Has This Been Tested? - Manual - Cypress ### Test Plan > Add Testsmith test cases links that relate to this PR ### Issues raised during DP testing > Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR) ## Checklist: ### Dev activity - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag ### QA activity: - [ ] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test
  • Compare 15 commits »

6 hours ago

tongchong synced commits to feat/action-selector at tongchong/appsmith from mirror

6 hours ago

tongchong synced commits to feat/20642-file-upload-bloat-removal at tongchong/appsmith from mirror

  • 4a72440e37 Tests and conforming payload to expected client payload

6 hours ago

tongchong synced commits to bundle-optimization at tongchong/appsmith from mirror

  • d497c7db25 Merge branch 'release' into bundle-optimization
  • adb6938e88 fix: keep sidebar open when any context menu is open (#21777)
  • 778922bd74 chore: Updating tests which will fail on app invites getting merged (#21949) ## Description > Updating tests which will fail when app invites are merged on EE. Along with that, updating permission check for the invite modal content on app level. Fixes [#21952](https://github.com/appsmithorg/appsmith/issues/21952) ## Type of change - Chore (housekeeping or task changes that don't impact user perception) ## How Has This Been Tested? > Ran the cypress tests on local and it works fine. - Manual - Cypress ## Checklist: ### Dev activity - [x] My code follows the style guidelines of this project - [x] I have performed a self-review of my own code - [x] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [x] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag ### QA activity: - [ ] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test
  • dffbf3d97c ci: Adding Identified Flaky tests app link into Git comments section in a PR (#21966) ## Description - This PR adds the Flaky list application link into the git comments section on completion of the run, so the PR owners have easy access to it. ## Type of change - Yml file update ## Checklist: ### QA activity: - [X] Added Test Plan Approved label after reviewing all changes
  • 63f63365d9 fix: Omnibar not showing recent entities (#21784) ## Description The Omnibar is supposed to update its order to ensure recent items are listed first. This stopped working at some time and this PR fixes it > Omnibar will list recently accessed items first Fixes #18963 ## Type of change - Bug fix (non-breaking change which fixes an issue) ## How Has This Been Tested? - Manual - Cypress ### Test Plan > Add Testsmith test cases links that relate to this PR ### Issues raised during DP testing > Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR) ## Checklist: ### Dev activity - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag ### QA activity: - [ ] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test
  • Compare 8 commits »

6 hours ago

tongchong synced commits to master at tongchong/amis from mirror

17 hours ago

tongchong synced commits to master at tongchong/phaser from mirror

1 day ago

tongchong synced commits to main at tongchong/lowcode-engine from mirror

1 day ago