-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
refactor: queries loading order and wrapper order #3603
Conversation
This pull request is automatically being deployed by Amplify Hosting (learn more). |
Pull Request Validation ReportThis comment is automatically generated by Conventional PR Whitelist Report
Result Pull request does not satisfy any enabled whitelist criteria. Pull request will be validated. Validation Report
Result Pull request satisfies all enabled pull request rules. Last Modified at 28 Aug 24 19:06 UTC |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
…efore loading the router
… drop to improve test suite cleanliness and maintainability.
…tionality 🔧 (auto-save-off.spec.ts): add click event for "Save And Exit" button 🔧 (dragAndDrop.spec.ts): change dispatchEvent to getByTestId and add assertions for specific text visibility 🔧 (store-shard-3.spec.ts): increase timeout for page.waitForTimeout to improve test reliability
The code changes include adding the functionality for the replace button on the main page. This allows users to replace a flow or a component. The replace button is now visible on the page, and clicking on it triggers the appropriate action. Recent user commits: - test: improve timeout for page.waitForSelector in auto-save-off.spec.ts - ✅ (folders.spec.ts): update test description to be more descriptive and accurate - ✨ (folders.spec.ts): add test for adding folder by drag and drop functionality - 🔧 (auto-save-off.spec.ts): add click event for "Save And Exit" button - 🔧 (dragAndDrop.spec.ts): change dispatchEvent to getByTestId and add assertions for specific text visibility - 🔧 (store-shard-3.spec.ts): increase timeout for page.waitForTimeout to improve test reliability Recent repository commits: - test: improve timeout for page.waitForSelector in auto-save-off.spec.ts - ✅ (folders.spec.ts): update test description to be more descriptive and accurate - ✨ (folders.spec.ts): add test for adding folder by drag and drop functionality - 🔧 (auto-save-off.spec.ts): add click event for "Save And Exit" button - 🔧 (dragAndDrop.spec.ts): change dispatchEvent to getByTestId and add assertions for specific text visibility - 🔧 (store-shard-3.spec.ts): increase timeout for page.waitForTimeout to improve test reliability - Fixed flow dropping to another folder - 📝 (folders.spec.ts): remove unused test for adding folder by drag and drop to improve test suite cleanliness and maintainability. - Made loading only exist in one place - Fixed order of wrappers in order for Auth and API context to have access to router - Made ProtectedRoute refresh on authentication change - Updated App.tsx with new gets and configurations and added a loader before loading the router - Updated other queries with the new useQueryFunctionType type
8ea3e70
to
70975e7
Compare
This pull request includes several commits that refactor various components in the codebase. The
ComponentsComponent
andheaderTabsComponent
have been refactored to receive information from their parent components. TheMyCollectionComponent
now loads folders. Theget config
API call has been updated to update everything. TheApp.tsx
file has been updated with new gets and configurations, and a loader has been added before loading the router. Additionally, a loading page has been added.