Skip to content
This repository was archived by the owner on Dec 17, 2024. It is now read-only.

Commit 015d588

Browse files
committed
avoid duplicate undeployed warnings in wskflow
Fixes #886
1 parent a6d4a57 commit 015d588

File tree

1 file changed

+31
-13
lines changed

1 file changed

+31
-13
lines changed

app/plugins/modules/wskflow/lib/fsm2graph.js

Lines changed: 31 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -583,20 +583,38 @@ function fsm2graph(ir, containerElement, acts, options){
583583
if (notDeployed.length > 0 && !activations) {
584584
const container = document.querySelector('#sidecar .sidecar-header .sidecar-header-secondary-content .custom-header-content')
585585
if (container) {
586-
const message = document.createElement('div'),
587-
warning = document.createElement('strong'),
588-
text = document.createElement('span'),
589-
examples = document.createElement('span')
590-
591-
message.appendChild(warning)
592-
message.appendChild(text)
593-
message.appendChild(examples)
594-
container.appendChild(message)
586+
const css = {
587+
message: 'wskflow-undeployed-action-warning',
588+
text: 'wskflow-undeployed-action-warning-text',
589+
examples: 'wskflow-undeployed-action-warning-examples'
590+
}
591+
let message = container.querySelector(`.${css.message}`),
592+
text, examples
593+
594+
if (!message) {
595+
const message = document.createElement('div'),
596+
warning = document.createElement('strong')
597+
598+
text = document.createElement('span')
599+
examples = document.createElement('span')
600+
601+
message.className = css.message
602+
text.className = css.text
603+
examples.className = css.examples
604+
605+
message.appendChild(warning)
606+
message.appendChild(text)
607+
message.appendChild(examples)
608+
container.appendChild(message)
595609

596-
warning.className = 'red-text'
597-
examples.className = 'deemphasize deemphasize-partial left-pad'
598-
599-
warning.innerText = 'Warning: '
610+
warning.className = 'red-text'
611+
examples.className = 'deemphasize deemphasize-partial left-pad'
612+
613+
warning.innerText = 'Warning: '
614+
} else {
615+
text = message.querySelector(`.${css.text}`)
616+
examples = message.querySelector(`.${css.examples}`)
617+
}
600618

601619
const actionStr = notDeployed.length === 1 ? 'action' : 'actions'
602620
text.innerText = `This composition depends on ${notDeployed.length} undeployed ${actionStr}`

0 commit comments

Comments
 (0)