-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cromwell metadata params [risk: low] #1456
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@@ -140,7 +141,7 @@ | |||
(set (keys query-parameters))) | |||
(str "Malformed query parameters: " query-parameters)) | |||
{:path (str "/workspaces/" (id-path workspace-id) "/entityQuery/" type "?" | |||
(clojure.string/join "&" (keep (fn [[k v]] | |||
(string/join "&" (keep (fn [[k v]] | |||
(some->> v str not-empty (str k "="))) | |||
query-parameters))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reindent
(links/create-internal {:onClick #(swap! state update :expanded? not)} | ||
(if expanded? "Hide" "Show"))) | ||
;; use expanded? to show/hide the WorkflowTimingDiagram component via the DOM | ||
[:div {:myattr "whereami" :style {:display (if expanded? "block" "none")}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
debugging artifact?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hahaha yes thank you for the catch.
DataBiosphere/firecloud-app#213
DataBiosphere/firecloud-app#214
when making requests to the Cromwell workflow metadata API (via orch and rawls), specify only those
includeKey
values the UI needs. This hint allows Cromwell to do only the work that's truly necessary, and keeps the response sizes smaller by eliminating cruft.additionally, don't load the data needed to power the timing diagram when first viewing a workflow's details, since the timing diagram is collapsed by default. Once the user clicks to expand the timing diagram, make a second ajax call to get the data it needs. This keeps the initial page much smaller and only gets timing data when absolutely necessary.