NYCPlanning / labs-community-profiles

A site for exploring indicators, data, and information for NYC community districts
https://communityprofiles.planning.nyc.gov/
Other
52 stars 9 forks source link

Task/9261 update app dependencies #660

Closed horatiorosa closed 2 years ago

horatiorosa commented 2 years ago

Changes Proposed:

minimum required changes to run app locally

Closes AB#9261

netlify[bot] commented 2 years ago

Deploy Preview for labs-communityprofiles ready!

Name Link
Latest commit 0c9c3b7b1dbecb229cea673b8234a8df1464753d
Latest deploy log https://app.netlify.com/sites/labs-communityprofiles/deploys/62b6208bf363970009c79d02
Deploy Preview https://deploy-preview-660--labs-communityprofiles.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] commented 2 years ago

Deploy Preview for community-profiles-new-data ready!

Name Link
Latest commit 0c9c3b7b1dbecb229cea673b8234a8df1464753d
Latest deploy log https://app.netlify.com/sites/community-profiles-new-data/deploys/62b6208b85e1130009bb5fdd
Deploy Preview https://deploy-preview-660--community-profiles-new-data.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

rtblair commented 2 years ago

@horatiorosa Please consider updating the readme if you discovered any undocumented steps for getting the app to run locally.

horatiorosa commented 2 years ago

Looks good to me. The only issue I can find is the Land Use Applications section does not load; that issue exists in production and is not introduced by this update. Please just check and make sure that there is a task in ADO for fixing that bug.

Thanks for the 👀👀. Yep, that was the BUG819 I was going to work on but had to roll in your updates first in a new branch since I cannot use the develop branch.