search

Swen670Grp1 / cytoscape-platform-tests-js

Reveal.js testing webapp for running platform tests on Cytoscape via CyREST calls
http://chianti.ucsd.edu/~bsettle/cytoscape_platform_test
MIT License
0 stars 0 forks source link

New slide creation #42

Open skatulich opened 5 years ago

skatulich commented 5 years ago

All updates have been made except for the Session.js file does not seem to be valid. There are lots of semi colons missing throughout the javascript. I added the develop JS files and now have an error for UAParser not being defined and "style" not being defined in main.js.

skatulich commented 5 years ago

Can you please update your reviews so this can be merged in?

skatulich commented 5 years ago

Please accept so I can merge.

kouissar commented 5 years ago

I tested it however I feel this is a regression vs the working version we have in develop. As I shared with Kemari, It is too high of a risk to introduce such big of a change that might render the app unusable at this late stage. I suggest we finalize develop and test it so at least we have a working model with few working and helpful features that can be augmented by Cytoscape team or Spring class. This is great work though, I would keep the configuration changes but cancel the the UI or design changes.