Improve monitor stability and fix exception on startup
Closes #29 (closed)
Merge request reports
Activity
added Bugfix statusSelected for Development labels
assigned to @gzi
requested review from @dugganjw
Yeah, so if it doesn't connect to Galaxy on startup (which should be the standard flow, since otherwise you have to set the API_KEY env var), then it will attempt to connect to Galaxy after you log in. If it fails to connect after you log in, we could implement like an alert or popup telling the user to refresh or something.
Edited by Cage, GregoryMakes sense. Just for my information, is the trigger then when Apache creates a new session? I guess one potential concern there is what happens when a user's session is preserved (maybe we don't need to do this if we can auto-login users). Not relevant to this MR, but something to think about with the rest of the Apache setup.
If you want to show an alert, then I'd recommend a VBanner underneath the app bar. The Vuetify docs are down, but I'll follow up with a link once they are up again. Feel free to just create a separate issue and assign it to me, if you'd prefer.
Edited by Duggan, Johnhttps://vuetifyjs.com/en/components/banners/#sticky Here's an example of what it could look like. Would recommend setting
bg_color='error'
,classes='text-center'
,density='compact'
, andsticky=True
. Haven't tested it but I think that would look good.
removed statusSelected for Development label
mentioned in commit bfef2405