tongchong
  • Joined on Dec 09, 2022
Loading Heatmap…

tongchong synced commits to svelte-5-adapter at tongchong/query from mirror

1 hour ago

tongchong synced commits to staleTime-static at tongchong/query from mirror

1 hour ago

tongchong synced commits to main at tongchong/query from mirror

  • 6a52926562 test(react-query): initiate queryClient in beforeEach (#9142)

1 hour ago

tongchong synced commits to chore/run-behviour-with-ff at tongchong/appsmith from mirror

  • 2d7ea5f166 Merge branch 'feat/reactive-actions-run-behaviour' of https://github.com/appsmithorg/appsmith into chore/run-behviour-with-ff
  • f34a597e6d feat: Updated references of all executeOnLoad to runBehaviour (#40361) ## Description > [!TIP] > _Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team)._ > > _Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR._ Fixes https://github.com/appsmithorg/appsmith/issues/40308, https://github.com/appsmithorg/appsmith/issues/40306, https://github.com/appsmithorg/appsmith/issues/40309, _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.All" ### :mag: Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/14906330551> > Commit: 9817e40e56b0d1143b7568cbb2272f0618404aac > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=14906330551&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.All` > Spec: > <hr>Thu, 08 May 2025 13:52:08 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No --------- Co-authored-by: “sneha122” <“sneha@appsmith.com”> Co-authored-by: nilansh <nilansh@appsmith.com>
  • bf058d007d ci: Fixing partial export spec (#40611) ## Description Fixing partial export spec Fixes [#39833](https://github.com/appsmithorg/appsmith/issues/39833) ## Automation /ok-to-test tags="" ### :mag: Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!CAUTION] > If you modify the content in this section, you are likely to disrupt the CI result for your PR. <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No
  • d0a1e83ff2 Merge branch 'release' of https://github.com/appsmithorg/appsmith into feat/reactive-actions-run-behaviour
  • ee2aeca9e7 Merge branch 'feat/reactive-actions-run-behaviour' of https://github.com/appsmithorg/appsmith into feat/reactive-actions-run-behaviour
  • Compare 11 commits »

1 day ago

tongchong synced commits to chore/remove-unnecessary-code at tongchong/appsmith from mirror

  • 164db73f8c minor refactoring
  • 609b5fe605 Merge branch 'chore/run-behviour-with-ff' of https://github.com/appsmithorg/appsmith into chore/remove-unnecessary-code
  • 2d7ea5f166 Merge branch 'feat/reactive-actions-run-behaviour' of https://github.com/appsmithorg/appsmith into chore/run-behviour-with-ff
  • Compare 3 commits »

1 day ago

tongchong synced commits to main at tongchong/query from mirror

  • 81aa02bc18 release: v5.75.8
  • 4b0f79fb4b fix(angular-query): allow providing on element injectors (#9134)
  • b1e0c9fd28 release: v5.75.7
  • 07cbfe1b36 refactor(react-query): exchange react-query noop to query-core noop (#9122) * refactor: exchange react-query noop to query-core noop * ci: apply automated fixes * fix: resolve import error --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
  • Compare 4 commits »

2 days ago

tongchong synced commits to main at tongchong/bootstrap from mirror

  • ac0e962d32 Drop ununsed `@types/google.analytics` dev dependency and bump `@types` dev dependencies (#41456) Co-authored-by: Christian Oliff <christianoliff@pm.me>

2 days ago

tongchong synced commits to chore/remove-unnecessary-code at tongchong/appsmith from mirror

2 days ago

tongchong synced commits to release at tongchong/appsmith from mirror

  • 7322d72e63 fix: enhance datasource structure retrieval in useTableOrSpreadsheet hook (#40612) ## Description <ins>Problem</ins> App was crashing when the data source configuration was undefined or in an error state. <ins>Root cause</ins> The code attempted to access properties from an undefined data source config (e.g., MongoDB), leading to runtime errors. <ins>Solution</ins> This PR handles adding an optional check before accessing the data source config to prevent crashes when the config is undefined or in an error state. Fixes #`Issue Number` _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.JSONForm, @tag.PropertyPane, @tag.GenerateCRUD, @tag.Datasource, @tag.Sanity, @tag.Widget" ### :mag: Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/14905964982> > Commit: bcd0f9d2d009bcf07bb10afda1a30ff1a93bc9ae > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=14905964982&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.JSONForm, @tag.PropertyPane, @tag.GenerateCRUD, @tag.Datasource, @tag.Sanity, @tag.Widget` > Spec: > <hr>Thu, 08 May 2025 13:32:39 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Bug Fixes** - Improved stability when accessing data source tables, preventing potential errors if data source information is missing. <!-- end of auto-generated comment: release notes by coderabbit.ai -->

2 days ago

tongchong synced commits to master at tongchong/appsmith from mirror

  • 9cfb1542d9 Merge pull request #40621 from appsmithorg/release 09/05 Daily Promotion
  • 1c531661c7 fix: improve infinite scroll reset logic in TableWidgetV2 (#40609) ## Description <ins>Problem</ins> Table would show inconsistent views or appear empty when infinite scroll setting changed or component height was modified. <ins>Root cause</ins> Page number was not reset properly during changes to infinite scroll or component height, causing the table to render an incorrect or empty state. <ins>Solution</ins> This PR handles improving the reset logic for the table when infinite scroll is enabled or component height changes. It refactors the reset conditions, introduces variables to track relevant state changes, and ensures the table resets correctly to maintain consistent behavior. Fixes #40411 _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.Table" ### :mag: Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/14903775919> > Commit: efe1c54169d21708cb519e749047ff207fffeb0e > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=14903775919&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.Table` > Spec: > <hr>Thu, 08 May 2025 10:36:21 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Refactor** - Improved the logic for resetting the table when infinite scroll is enabled or when the component height changes, resulting in clearer and more consistent behavior. No visible changes to features or exported entities. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
  • 0e5252c30c chore: fix container query bug in ai chat widget (#40607) /ok-to-test tags="@tag.Anvil" <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/14900550928> > Commit: a3a3579aaac44d2cfdfa956d46024145c4bfdad9 > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=14900550928&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.Anvil` > Spec: > <hr>Thu, 08 May 2025 07:22:28 UTC <!-- end of auto-generated comment: Cypress test results --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Style** - Introduced a named container ("flex-container") to relevant CSS classes for improved container query support. - Updated container queries to explicitly reference the new container name, enabling more precise and scoped responsive styling. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
  • 8a67948f1b chore: Update the carbon modal ux (#40599) /ok-to-test tags="@tag.Settings" Corresponding CE PR for https://github.com/appsmithorg/appsmith-ee/pull/7416 <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Refactor** - Updated internal logic related to AI Agent template imports to streamline behavior. No visible changes to user-facing features. <!-- end of auto-generated comment: release notes by coderabbit.ai --> <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/14899555055> > Commit: f0d187eb480064a68fe17c1fdc83112d6695f4b9 > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=14899555055&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.Settings` > Spec: > <hr>Thu, 08 May 2025 06:09:23 UTC <!-- end of auto-generated comment: Cypress test results -->
  • Compare 4 commits »

2 days ago

tongchong synced commits to chore/run-behviour-with-ff at tongchong/appsmith from mirror

  • 33af677199 chore: added run behaviour for js actions

2 days ago

tongchong synced commits to main at tongchong/query from mirror

  • cb40661209 docs(comparison): mark RTK Query as supporting infinite queries (#9120) RTK Query now supports infinite queries as introduced in Redux Toolkit v2.6.0. This feature allows efficient pagination handling through useInfiniteQuery, with built-in support for fetching next/previous pages and managing paginated cache state. Docs: https://redux-toolkit.js.org/rtk-query/usage/infinite-queries Release: https://github.com/reduxjs/redux-toolkit/releases/tag/v2.6.0
  • 3da0c7ecdd release: v5.75.6
  • 295a5d586f refactor: integration should throw error (#9130) * feat(query-core): made "shouldThrowError" function * refactor(react-query): apply "shouldThrowError" function in react-query package * refactor(angular-query-experimental): apply "shouldThrowError" function * refactor(solid-query): apply "shouldThrowError" function * refactor(vue-query): apply "shouldThrowError" function * test: add shouldThrowError function test
  • e0ea8f22f2 docs: remove unnecessary text (#9131)
  • Compare 4 commits »

2 days ago

tongchong synced commits to feat/reactive-actions-run-behaviour at tongchong/appsmith from mirror

  • f34a597e6d feat: Updated references of all executeOnLoad to runBehaviour (#40361) ## Description > [!TIP] > _Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team)._ > > _Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR._ Fixes https://github.com/appsmithorg/appsmith/issues/40308, https://github.com/appsmithorg/appsmith/issues/40306, https://github.com/appsmithorg/appsmith/issues/40309, _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.All" ### :mag: Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/14906330551> > Commit: 9817e40e56b0d1143b7568cbb2272f0618404aac > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=14906330551&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.All` > Spec: > <hr>Thu, 08 May 2025 13:52:08 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No --------- Co-authored-by: “sneha122” <“sneha@appsmith.com”> Co-authored-by: nilansh <nilansh@appsmith.com>

3 days ago

tongchong synced commits to main at tongchong/query from mirror

  • 57251a4e24 feature(angular-query): support providing QueryClient via an InjectionToken (#9128) This is useful for example in thin app shell applications where feature teams provide their dependencies on lazy loaded route providers while still allowing to easily share a QueryClient. This way the main bundle for the app shell does not include TanStack Query.
  • 8f092f9e4a ci: apply automated fixes
  • 6ee1e666fe Update README.md
  • 37d8eb6ff9 ci: apply automated fixes
  • 32414aab2f Update README.md
  • Compare 6 commits »

3 days ago

tongchong synced commits to master at tongchong/amis from mirror

3 days ago

tongchong synced commits to release at tongchong/appsmith from mirror

  • 1c531661c7 fix: improve infinite scroll reset logic in TableWidgetV2 (#40609) ## Description <ins>Problem</ins> Table would show inconsistent views or appear empty when infinite scroll setting changed or component height was modified. <ins>Root cause</ins> Page number was not reset properly during changes to infinite scroll or component height, causing the table to render an incorrect or empty state. <ins>Solution</ins> This PR handles improving the reset logic for the table when infinite scroll is enabled or component height changes. It refactors the reset conditions, introduces variables to track relevant state changes, and ensures the table resets correctly to maintain consistent behavior. Fixes #40411 _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.Table" ### :mag: Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/14903775919> > Commit: efe1c54169d21708cb519e749047ff207fffeb0e > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=14903775919&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.Table` > Spec: > <hr>Thu, 08 May 2025 10:36:21 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Refactor** - Improved the logic for resetting the table when infinite scroll is enabled or when the component height changes, resulting in clearer and more consistent behavior. No visible changes to features or exported entities. <!-- end of auto-generated comment: release notes by coderabbit.ai -->

3 days ago

tongchong synced commits to feat/update-all-references-be at tongchong/appsmith from mirror

  • 9817e40e56 fix: executeOnLoad made internal in actionviewCEDTO
  • c57ff4751c Merge branch 'feat/reactive-actions-run-behaviour' into feat/update-all-references-be
  • bf058d007d ci: Fixing partial export spec (#40611) ## Description Fixing partial export spec Fixes [#39833](https://github.com/appsmithorg/appsmith/issues/39833) ## Automation /ok-to-test tags="" ### :mag: Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!CAUTION] > If you modify the content in this section, you are likely to disrupt the CI result for your PR. <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No
  • d0a1e83ff2 Merge branch 'release' of https://github.com/appsmithorg/appsmith into feat/reactive-actions-run-behaviour
  • ee2aeca9e7 Merge branch 'feat/reactive-actions-run-behaviour' of https://github.com/appsmithorg/appsmith into feat/reactive-actions-run-behaviour
  • Compare 12 commits »

3 days ago

tongchong synced commits to feat/reactive-actions-run-behaviour-be at tongchong/appsmith from mirror

3 days ago

tongchong synced commits to feat/reactive-actions-run-behaviour at tongchong/appsmith from mirror

  • bf058d007d ci: Fixing partial export spec (#40611) ## Description Fixing partial export spec Fixes [#39833](https://github.com/appsmithorg/appsmith/issues/39833) ## Automation /ok-to-test tags="" ### :mag: Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!CAUTION] > If you modify the content in this section, you are likely to disrupt the CI result for your PR. <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No
  • d0a1e83ff2 Merge branch 'release' of https://github.com/appsmithorg/appsmith into feat/reactive-actions-run-behaviour
  • ee2aeca9e7 Merge branch 'feat/reactive-actions-run-behaviour' of https://github.com/appsmithorg/appsmith into feat/reactive-actions-run-behaviour
  • 1c531661c7 fix: improve infinite scroll reset logic in TableWidgetV2 (#40609) ## Description <ins>Problem</ins> Table would show inconsistent views or appear empty when infinite scroll setting changed or component height was modified. <ins>Root cause</ins> Page number was not reset properly during changes to infinite scroll or component height, causing the table to render an incorrect or empty state. <ins>Solution</ins> This PR handles improving the reset logic for the table when infinite scroll is enabled or component height changes. It refactors the reset conditions, introduces variables to track relevant state changes, and ensures the table resets correctly to maintain consistent behavior. Fixes #40411 _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.Table" ### :mag: Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/14903775919> > Commit: efe1c54169d21708cb519e749047ff207fffeb0e > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=14903775919&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.Table` > Spec: > <hr>Thu, 08 May 2025 10:36:21 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Refactor** - Improved the logic for resetting the table when infinite scroll is enabled or when the component height changes, resulting in clearer and more consistent behavior. No visible changes to features or exported entities. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
  • 0e5252c30c chore: fix container query bug in ai chat widget (#40607) /ok-to-test tags="@tag.Anvil" <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/14900550928> > Commit: a3a3579aaac44d2cfdfa956d46024145c4bfdad9 > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=14900550928&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.Anvil` > Spec: > <hr>Thu, 08 May 2025 07:22:28 UTC <!-- end of auto-generated comment: Cypress test results --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Style** - Introduced a named container ("flex-container") to relevant CSS classes for improved container query support. - Updated container queries to explicitly reference the new container name, enabling more precise and scoped responsive styling. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
  • Compare 9 commits »

3 days ago

tongchong synced commits to springboot3 at tongchong/RuoYi from mirror

3 days ago