Refreshing after editing constructs is causing large instruments is slow to load

CLOSER-Cohorts / archivist

Archivist is designed to be an easy to use editor that allows documention of questionnaires and data in DDI-Lifecycle

MIT License

17 stars 8 forks source link

Refreshing after editing constructs is causing large instruments is slow to load #363

Closed HayleyMills closed 4 years ago

HayleyMills commented 5 years ago

JS tested on USoc instance. When you update a condition and save it, then the qc labels inside the condition disappear on the screen. A refresh appears to bring them back. However this refresh is taking a long time for large instruments (even when dynos turned up).

simonreed commented 5 years ago

Just to clarify can you confirm what page this is on with an example URL, preferably on staging?

HayleyMills commented 5 years ago

https://closer-archivist-staging.herokuapp.com/instruments/bcs9/build/constructs

HayleyMills commented 4 years ago

Tested using https://closer-archivist-staging.herokuapp.com/instruments/bcs9/build/constructs. Changed c_ifcurrentresidence to c_ifcurrentresidence_test and it is now much quicker to refresh. Is it possible to retain the display of the constructs inside of the condition, so we don't have to refresh?

HayleyMills commented 4 years ago

This issue has been resolved, there is another issue mentioned in the above comment which I have made a new issue for #418