-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Fix : Caching the foreign key column options #9828
Merged
akshaysasidrn
merged 11 commits into
feature/infinitescroll-for-foreignkeydata-tjdb
from
manish/fk-options-loading
May 30, 2024
Merged
Fix : Caching the foreign key column options #9828
akshaysasidrn
merged 11 commits into
feature/infinitescroll-for-foreignkeydata-tjdb
from
manish/fk-options-loading
May 30, 2024
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
…ardless of whether they resolve or reject.
Deployment: https://tooljet-pr-9828.onrender.com |
ganesh8056
reviewed
May 23, 2024
ganesh8056
reviewed
May 23, 2024
ganesh8056
reviewed
May 23, 2024
ganesh8056
reviewed
May 23, 2024
Deployment: https://tooljet-pr-9828.onrender.com |
akshaysasidrn
requested changes
May 23, 2024
frontend/src/Editor/QueryManager/QueryEditors/TooljetDatabase/SelectBox.jsx
Outdated
Show resolved
Hide resolved
akshaysasidrn
approved these changes
May 23, 2024
…nish/fk-options-loading
akshaysasidrn
merged commit May 30, 2024
55d5753
into
feature/infinitescroll-for-foreignkeydata-tjdb
5 of 7 checks passed
akshaysasidrn
added a commit
that referenced
this pull request
Jun 4, 2024
…oolJet database. (#9785) * feat: Added a support for infinite scroll on searching foreign key data * feat: added debounce for search in infinite scroll and refactored the infinite scroll logic on listing Foreign key data * fix: while searching foriegn key values in the drop down of tooljet database for varchar datatype possible search results will be returned and for other datatype strict matching is followed * fix: review comments for optional chaining has been fixed * fix: typo error has been fixed * fix: client search on foreign key drop down is disabled * stylefix: menu height is fixed now for foreign key drop down * fix: foreign key drop down scroll bar moves to top on each render fixed * Fix : Caching the foreign key column options (#9828) * Draft : Caching the fk options * Refactored the code and use promise.all * Used Promise.allSettled to ensure that all promises are processed regardless of whether they resolve or reject. * Changed currentColumn to foreignKey * Removed console * Updated default value of cachedOptions state to empty object * Updated the code with suggestion to uplift check condition * Change the emptyError copyright to No values found * Bug fixed : on mount api request failing * Refactored code from calculations to data by removing optional chaning * Fix: Remove null values from the foreign key drop-down list (#9895) * fix: filtered out null values and brought empty string to top in the foreign key drop down * fix: review comments addressed * fix: while creating or editing FK relation the source column drop down will not list empty column names * stylefix: cell was not clickable on empty string * fix: empty values are not listed on Foreign key drop down used for cell edit * fix: empty string can be added as default in create table operation * Fixed : Fk options are not cached when source column name is updated --------- Co-authored-by: Akshay <[email protected]> Co-authored-by: Manish Kushare <[email protected]> Co-authored-by: manishkushare <[email protected]>
9 tasks
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.
Resolves #9745