cytoscape/cyREST

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

Closed this issue · 2 comments

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

We should either:

  • Notify users, right at the top of the Swagger UI (maybe include a reload button right on the page).
  • Disable the menu items leading to Swagger until all Apps are loaded.

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.

This is made largely irrelevant due to 2154baf

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 #85.

Additionally added to Swagger here: cc790db