-
Notifications
You must be signed in to change notification settings - Fork 56
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
Bug: Execution Page's back button returns Workflows route from Launch Plan route #patch #760
Bug: Execution Page's back button returns Workflows route from Launch Plan route #patch #760
Conversation
Thank you for opening this pull request! 🙌 These tips will help get your PR across the finish line:
|
cd6b82d
to
2154eab
Compare
Signed-off-by: Frank Flitton <fflitton@gmail.com>
2154eab
to
9c57fdf
Compare
Nice work adding the extra tests 💯 |
Codecov Report
@@ Coverage Diff @@
## master #760 +/- ##
=======================================
Coverage 65.90% 65.90%
=======================================
Files 489 489
Lines 11917 11918 +1
Branches 2198 2198
=======================================
+ Hits 7854 7855 +1
Misses 4063 4063
|
@@ -1,7 +1,7 @@ | |||
import * as React from 'react'; |
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.
I see in some places we use import * as React but in other we use import React from "react".
Probably we dont need the * as react anymore?
Signed-off-by: Frank Flitton <fflitton@gmail.com>
f6cd482
to
a191e04
Compare
Congrats on merging your first pull request! 🎉 |
Signed-off-by: Yee Hing Tong <wild-endeavor@users.noreply.github.com> ### Admin - v1.1.100 * Inject user identifier to ExecutionSpec by @ByronHsu in flyteorg/flyteadmin#549 * Fix flaky test by @eapolinario in flyteorg/flyteadmin#563 * Add oauth http proxy for external server & Extract email from azure claim by @ByronHsu in flyteorg/flyteadmin#553 * Remove single task execution default timeout by @hamersaw in flyteorg/flyteadmin#564 * Revert conditional setting of SecurityContext when launching security context by @wild-endeavor in flyteorg/flyteadmin#566 ### Console - v1.8.2 * Export Flytedecks support for TLRO by @james-union in flyteorg/flyteconsole#757 * fix: filter executions by version and name by @ursucarina in flyteorg/flyteconsole#758 * fix: task recent runs should filter by version by @ursucarina in flyteorg/flyteconsole#759 * Bug: Execution Page's back button returns Workflows route from Launch Plan route #patch by @FrankFlitton in flyteorg/flyteconsole#760 * chore: add item when mapped task by @jsonporter in flyteorg/flyteconsole#761 * Feature: Fullview Flyte Deck modal by @FrankFlitton in flyteorg/flyteconsole#764 ### Propeller - v1.1.90 * Add grpc plugin to loader.go by @pingsutw in flyteorg/flytepropeller#562
🎉 This PR is included in version 1.8.3 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
TL;DR
Fixes routing behaviour for back arrow button for Launch Plans workflow execution detail referred pages.
Type
Are all requirements met?
Complete description
Added
fromLink
to routerstate
to use existing logic to match other workflow tables.Follow-up issue
NA