search

YaleDHLab / nhba

A digital archive of New Haven's architecture
https://nhba.yale.edu
4 stars 4 forks source link

Housekeeping #332

Closed duhaime closed 6 years ago

duhaime commented 6 years ago

@spaceraccoon I think we should move to yarn.lock rather than the npm shrinkwrap. Also wanted to reflect the doco change and quash a warning in the current master that springs when attempting to connect with Mongo. If this looks good feel free to merge and drop the shrinkwrap...

spaceraccoon commented 6 years ago

Great, thanks! Yes, I also prefer to use yarn.

spaceraccoon commented 6 years ago

As a side note, I noticed some deprecation warnings - do we plan to upgrade to React 16 and a couple of the other dependencies?

duhaime commented 6 years ago

It would be awesome to move to 16. Would you have time for that? If not just let me know and we can find a solution...

duhaime commented 6 years ago

ping @spaceraccoon

spaceraccoon commented 6 years ago

Sure! I'm almost done with the urgent feature requests for this week and will look into it. @duhaime I know for the Confirm modal fragments would've been super useful.

duhaime commented 6 years ago

@spaceraccoon Yes I hear you. Fire at will when time permits...