Closed jimmyzhen closed 2 years ago
@cteng585 Thank you for reviewing the PR. I've addressed the bug pertinent to the first bullet point in your comments as well as committed the change into the PR.
In regards to the third bullet point in your comments, given the plan to re-implement the file-browsing UIs with the rat figure diagram, I think it may make sense to hold off and see whether it will remain an issue after the overhaul.
Everything looks great. Thanks for fixing the bug I mentioned in the first bullet.
Key Changes:
Technical Notes:
To spin up a local instance, please do the following:
node.js v14.18.0
to build the app and run the tests.rm -fr node_modules/
,rm yarn.lock
,yarn install
, andyarn sass
sequentially upon cloning/pulling this branch..env
file consisting of a number environment variables is also required at the root level. Reach out to me for details.yarn start
.