Closed Fatima-Bibi closed 2 weeks ago
[!CAUTION]
Review failed
The pull request is closed.
The pull request introduces multiple updates across various components within the multi-vendor admin system. Key changes include the addition of GraphQL mutation definitions for managing food, categories, and vendor entities, as well as updates to the App
, Addon
, Rider
, and Configuration
components to enhance error handling and user experience. Modifications to constants for server URLs and validation constraints are also included. These changes collectively maintain existing functionalities while improving state management, error handling, and UI responsiveness.
File Path | Change Summary |
---|---|
src/apollo/mutations.js |
Introduced GraphQL mutations for food, categories, and vendors; minor formatting change in saveSendGridApiKey . |
src/app.js |
Added offline detection with a retry button; new imports for MUI components; updated rendering logic for online/offline status. |
src/components/Addon/Addon.jsx |
Added console log for debugging; improved state management and error handling for addon mutations. |
src/components/Rider/Rider.jsx |
Introduced phoneErrorMessage state for phone validation; updated error handling for user feedback. |
src/config/constants.js |
Updated SERVER_URL and WS_SERVER_URL to point to a local server; previous URLs retained as comments. |
src/constraints/constraints.js |
Modified validation constraints for phone and tip fields; updated validateFuncForRider function. |
src/context/Configuration.js |
Enhanced error handling in ConfigurationProvider by ensuring configuration has a defined structure. |
src/index.js |
Added WebSocket link for GraphQL subscriptions; encapsulated request handling with token authorization. |
src/views/Users.jsx |
Minor adjustments to error handling and corrected typo in className prop. |
src/components/Footers/AdminFooter.jsx |
Updated href for "About Us" link to point to a new URL. |
Configuration.js
regarding the initialization of the configuration
variable are related to the main PR's focus on managing configurations, particularly with the saveEmailConfiguration
and other configuration mutations.Configuration.js
file, enhancing the logic for setting the configuration
object, which aligns with the main PR's updates to configuration-related mutations.Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Name | Link |
---|---|
Latest commit | cc1f3fd12524ad9654eeb7025fa322adc58b0da9 |
Latest deploy log | https://app.netlify.com/sites/polite-fairy-234917/deploys/67344ecd1f5d3700086e8cdf |
Name | Link |
---|---|
Latest commit | cc1f3fd12524ad9654eeb7025fa322adc58b0da9 |
Latest deploy log | https://app.netlify.com/sites/cheery-zabaione-34f12e/deploys/67344ecd5bb6ec0009ef169d |
Deploy Preview | https://deploy-preview-657--cheery-zabaione-34f12e.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.