PulseTile / PulseTile-RA

PulseTile exploration with React Admin
Apache License 2.0
5 stars 6 forks source link

PulseTile RA - troubleshooting build files - dependency versions +/- minification #62

Open tony-shannon opened 5 years ago

tony-shannon commented 5 years ago

as result of discussion on UI bugs noted here https://github.com/RippleOSI/NHS_Ed_Scotland_Open_Platform/issues/53

we need to investigate the root cause of some of this bizarre behaviour at the UI

Few points; Local dev seems ok according to Bogdan Remote deployment is where we are seeing the issue, on DO VPS machine So likely to be an issue with the build/minification process

Few points to note

React Admin currently uses version 1.5.1 of Material UI, not the latest version 3.8.9, so that needs checking for consistency .. can be checked in node-modules directory

Other dependencies eg here need to be updated; https://github.com/PulseTile/PulseTile-RA/blob/scotland/package.json eg https://github.com/PulseTile/PulseTile-RA/blob/scotland/package.json#L32 "react-scripts": "1.1.4",

As the build process involves Create React App, which involves react-scripts this is now of of date (latest version of react-scripts is 2.1.8) . Important to note that use of ^ would help ensure latest version is involved https://michaelsoolee.com/npm-package-tilde-caret/ So recommend to use "react-scripts": "^1.1.4" or perhaps ,"react-scripts": "^2.1.8"

Also note that the versions here are out of sync and out of date; https://github.com/PulseTile/PulseTile-RA/blob/scotland/package.json#L22 https://github.com/PulseTile/PulseTile-RA/blob/scotland/package.json#L24 suggest both are reconciled to the same version "^2.7.3"

If these are updated, it should help a lot

Then

BogdanScherban commented 5 years ago

This issue was solved by dependencies updates.