search

cytoscape / cyREST

Core App: REST API module for Cytoscape
https://github.com/cytoscape/cyREST/wiki
MIT License
30 stars 13 forks source link

Notify users that all Apps are not loaded in Swagger, or disable menu items #80

Closed dotasek closed 5 years ago

dotasek commented 6 years ago

Sometimes, users can access the Swagger UI before all apps are loaded, resulting in some longer loading apps being unavailable.

We should either:

Out of these two, I favour the second. I feel like the all-apps-loaded flag might malfunction, and take away the ability to examine things via the Swagger UI.

However, the Swagger UI would still be available via direct HTML access in this case.

dotasek commented 6 years ago

This is made largely irrelevant due to https://github.com/cytoscape/cyREST/commit/2154bafee5228a478100e9fea78d54f4486b3514

Swagger is provided by the CyREST server, and now that server waits until all apps are loaded before starting.

The only possible edge case for this is when an App could be started by the App Manager. For this case, I made https://github.com/cytoscape/cyREST/issues/85.

dotasek commented 5 years ago

Additionally added to Swagger here: cc790db