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

Data view loading optimizations #148

Merged
merged 1 commit into from
Mar 16, 2023
Merged

Conversation

garrylachman
Copy link
Owner

@garrylachman garrylachman commented Mar 16, 2023

close #147

PR summary:

  • Refactored code in multiple files to improve performance and reliability
  • Implemented ReactDataGrid to display data
  • Added unique IDs to menu items and set isLazy to fixed
  • Added text to loading spinner

@garrylachman garrylachman self-assigned this Mar 16, 2023
@garrylachman garrylachman linked an issue Mar 16, 2023 that may be closed by this pull request
@@ -132,7 +132,7 @@ export const FilterBuilderWheresGroup: FC<FilterBuilderGroupProperties> = ({
index={index + 1}

Choose a reason for hiding this comment

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

GPT summary of 888ad2 - a9cde2:

  • Changed the loading spinner to include text "Working..."

@github-actions
Copy link

GPT summary of 09b632c:

Error: couldn't generate summary

PR summary so far:

  • Refactored code in multiple files to improve performance and reliability
  • Implemented ReactDataGrid to display data
  • Added unique IDs to menu items and set isLazy to fixed
  • Added text to loading spinner

@garrylachman garrylachman merged commit 338d784 into develop Mar 16, 2023
@garrylachman garrylachman deleted the 147-loading-view-lagging branch March 16, 2023 18:55
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.

Loading view lagging
1 participant