Closed ColinBuyck closed 2 months ago
Name | Link |
---|---|
Latest commit | 96592dd3a926217090c8e6c294fc89707a88c4ba |
Latest deploy log | https://app.netlify.com/sites/partners-bloom-dev/deploys/66c51c26b4ffcc00084c269c |
Deploy Preview | https://deploy-preview-4252--partners-bloom-dev.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.
Name | Link |
---|---|
Latest commit | 96592dd3a926217090c8e6c294fc89707a88c4ba |
Latest deploy log | https://app.netlify.com/sites/bloom-exygy-dev/deploys/66c51c262946500008c2481b |
Deploy Preview | https://deploy-preview-4252--bloom-exygy-dev.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.
This PR addresses #4248
Description
This PR adds feature flagging to hide or show the side nav and creates a custom call to reduce the load time for accounts with numerous applications.
Add SHOW_LOTTERY=TRUE to the public side
One note that you will see ApplicationsIndexEnum and ApplicationsFilterEnum are very similar but based on the current implementation requiring tabs, offline convo landed on using both would be best to maintain strong typing on the BE and easy tab selection based on index + better readability on the FE.
How Can This Be Tested/Reviewed?
This can be tested by signing into an account, applying to more than one listing, and seeing in the network tab that only one call is being made no matter how many listings there are. Additionally, you can rerun the application with the new feature flag set to false, and you will no longer see the Lottery run tab but everything else should work as expected.
Author Checklist:
yarn generate:client
and/or created a migration when requiredReview Process: