Skip to content

fix(ui): Multiple UI fixes#5498

Merged
simster7 merged 3 commits into
argoproj:masterfrom
simster7:tui
Mar 23, 2021
Merged

fix(ui): Multiple UI fixes#5498
simster7 merged 3 commits into
argoproj:masterfrom
simster7:tui

Conversation

@simster7
Copy link
Copy Markdown
Member

@simster7 simster7 commented Mar 23, 2021

Fixes #5496
Fixes #5494

Signed-off-by: Simon Behar <simbeh7@gmail.com>
Comment on lines +111 to +112
selectedPhases: phaseQueryParam.length > 0 ? (phaseQueryParam as WorkflowPhase[]) : savedOptions.selectedPhases,
selectedLabels: labelQueryParam.length > 0 ? (labelQueryParam as string[]) : savedOptions.selectedLabels,
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes #5496

Comment thread ui/src/app/shared/components/object-editor/object-editor.tsx Outdated
@simster7 simster7 marked this pull request as draft March 23, 2021 15:30
Comment thread ui/src/app/shared/components/object-editor/object-editor.tsx Outdated
Signed-off-by: Simon Behar <simbeh7@gmail.com>
@simster7 simster7 changed the title fix: Multiple UI fixes fix(ui): Multiple UI fixes Mar 23, 2021
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 23, 2021

Codecov Report

Merging #5498 (4688136) into master (be44ce9) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head 4688136 differs from pull request most recent head cf5506f. Consider uploading reports for the commit cf5506f to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5498      +/-   ##
==========================================
- Coverage   46.66%   46.64%   -0.03%     
==========================================
  Files         240      240              
  Lines       14985    14985              
==========================================
- Hits         6993     6990       -3     
- Misses       7093     7094       +1     
- Partials      899      901       +2     
Impacted Files Coverage Δ
workflow/controller/operator.go 70.25% <100.00%> (-0.28%) ⬇️
workflow/metrics/server.go 17.02% <0.00%> (+4.25%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update be44ce9...cf5506f. Read the comment docs.

Signed-off-by: Simon Behar <simbeh7@gmail.com>
return 'redirect=' + urlParams.get('redirect');
}
return '';
return 'redirect=' + window.location.origin + '/workflows';
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes #5494

@simster7 simster7 marked this pull request as ready for review March 23, 2021 17:55
@simster7 simster7 merged commit 4eb351c into argoproj:master Mar 23, 2021
@simster7 simster7 mentioned this pull request Mar 23, 2021
34 tasks
simster7 added a commit that referenced this pull request Mar 23, 2021
Signed-off-by: Simon Behar <simbeh7@gmail.com>
This was referenced Mar 29, 2021
@simster7 simster7 mentioned this pull request Apr 19, 2021
50 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Label choices aren't remembered When I first log in, I am logged in to a blank screen

2 participants