reduce UI ajax calls when switching tabs inside a workspace

[davidangb]

code in question:
https://github.com/broadinstitute/firecloud-ui/blob/develop/src/cljs/main/broadfcui/page/workspace/details.cljs#L168

The call to -refresh-workspace inside component-will-receive-props means that every time we switch tabs inside a workspace, we fire off two ajax calls - one to GET the workspace details, and another to checkBucketReadAccess. Both of these calls are relatively expensive.

We may have added this refresh solely to fix https://broadinstitute.atlassian.net/browse/GAWB-2485; see
broadinstitute/firecloud-ui#916. If this is the case, it's overkill - we can find a better way to handle refreshing after a clone, without requiring those ajax calls on each tab switch.

TODO:

• talk to the original developers to understand reasoning and implementation
• devise and test a fix