-
Notifications
You must be signed in to change notification settings - Fork 480
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 App.tsx
, Providers, Integrations; Remove usage on redux's states; Adds support for notifications to useQuery
and request
#6287
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Passing run #1282 ↗︎
Details:
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. |
App.tsx
, Providers, Integrations & Remove usage on redux's statesApp.tsx
, Providers, Integrations; Remove usage on redux's states; Adds support for notifications to useQuery
and request
@rithviknishad the cypress is failing look into it |
@rithviknishad the cypress is failing |
LGTM |
@rithviknishad We truly appreciate your efforts. Thank you for taking the time to contribute; this is a very valuable contribution to us 🥇. We always welcome your contribution 🙂, so feel free to contribute to anything anytime, and never lose that spirit of innovation 🙌. |
WHAT
🤖 Generated by Copilot at fca61b5
This pull request refactors the codebase to improve the organization, readability, and testability of the components. It extracts some logic from the
App
component into custom providers and routers, and moves and renames some components to new folders. It also updates the import paths and types for the config, currentUser, Sentry, Plausible, and HistoryAPIProvider components.Proposed Changes
App.tsx
, Providers, Integrations & Remove usage on redux's statesuseQuery
andrequest
to replacereact-redux
#6058useQuery
andrequest
reattempts
torequest
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist
HOW
🤖 Generated by Copilot at fca61b5
triggerGoal
function from Plausible component in various components (link, link, link, link, link)