allow users to choose viewed log - #174
Conversation
…w logs are displayed for. Currently limited to only successfull workflows.
|
I switched the jobsviewer to jobstable, as was having issues pass varibles multiple times, but also seem like now visits/(visit info) are defined on the app level. hope this is okay. |
There was a problem hiding this comment.
One of the best things about typescript is that it has types. I strongly suggest that you use them and, most importantly of all, avoid using the any type like the plague. I think you've created more work for yourself because you are manually checking the type when the language can do that for you.
yousefmoazzam
left a comment
There was a problem hiding this comment.
Thanks for this!
As mentioned in #174 (comment), we can tidy the nested if statements later when passing workflow info via job table row clicking is implemented.
8c2677e to
225a1fa
Compare
See #174 (comment) for information on further refactoring of the nested if statements once job table row clicking is implemented.
updating the logging component to allow users to choose which workflow logs are displayed for. Currently limited to only successfull workflows.