azavea / pwd-unitybar

Unified header for PWD Stormwater web apps
https://staging-pwd-unitybar.netlify.com/
Apache License 2.0
1 stars 2 forks source link

Create enhanced demo app with props toggles & set up Netlify staging site #65

Closed kellyi closed 5 years ago

kellyi commented 5 years ago

Overview

This PR adds a small demo application using material UI which enables users to toggle some of the application's simple props. It also configures the application to deploy via Netlify.

https://deploy-preview-65--staging-pwd-unitybar.netlify.com/

I left out some of the more complex props, including things requiring functions or lists, since it'd be hard to toggle those in a small UI.

Connects #61 Connects #64

Demo

https://deploy-preview-65--staging-pwd-unitybar.netlify.com/

Testing Instructions

kellyi commented 5 years ago

I pushed a last commit to drop an unused BundleAnalyzer plugin from the staging webpack build. In turn Netlify is now building this branch at

https://deploy-preview-65--staging-pwd-unitybar.netlify.com/

kellyi commented 5 years ago

Is there a way to prevent Webpack Bundle Analyzer from auto-launching a browser tab every time ./scripts/server is run?

Yeah, let me see if I can adjust this.

kellyi commented 5 years ago

046ddb4ff2348a24104e11a5ac567508e0cac126 adds a property to the development bundle analyzer server to prevent it from opening a new browser tab each time the server is launched.

kellyi commented 5 years ago

Weird! I'll keep a lookout for similar errors. The webpack dev server now has a couple processes running through it simultaneously including linting, Prettier, and hot module reloading -- so I assume those were somehow in conflict at whatever state your local setup was in at that moment?

kellyi commented 5 years ago

Thanks for reviewing!