Skip to content
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

feat: set jupyter server at first in Logs view (#1743) #2137

Merged
merged 3 commits into from
Nov 18, 2022

Conversation

andre-code
Copy link
Contributor

@andre-code andre-code commented Nov 10, 2022

PR to set Jupyter server at first in Logs view and don't show logs when there is no data case "download-image".

session logs

Closes #1743

/deploy #persist

@andre-code andre-code temporarily deployed to renku-ci-ui-2137 November 10, 2022 15:00 Inactive
@RenkuBot
Copy link
Contributor

You can access the deployment of this PR at https://renku-ci-ui-2137.dev.renku.ch

@andre-code andre-code force-pushed the 1743-show-jupyter-server-as-default branch from df0ee79 to 6fe10f2 Compare November 11, 2022 09:46
@andre-code andre-code temporarily deployed to renku-ci-ui-2137 November 11, 2022 09:46 Inactive
@andre-code andre-code force-pushed the 1743-show-jupyter-server-as-default branch from 6fe10f2 to f9ad36e Compare November 11, 2022 13:51
@andre-code andre-code changed the title WIP feat: set jupyter server at first in Logs view feat: set jupyter server at first in Logs view (#1743) Nov 11, 2022
@andre-code andre-code marked this pull request as ready for review November 11, 2022 13:56
@andre-code andre-code requested a review from a team as a code owner November 11, 2022 13:56
@andre-code andre-code temporarily deployed to renku-ci-ui-2137 November 11, 2022 13:56 Inactive
Copy link
Member

@lorenzo-cavazzi lorenzo-cavazzi left a comment

Choose a reason for hiding this comment

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

The code looks good!
I see the session logs acceptance test is failing, it probably requires tweaking it.

I noticed a bunch of visual details we could improve, but since they don't belong to this PR I made a new issue and added it to the sprint #2139

@andre-code andre-code temporarily deployed to renku-ci-ui-2137 November 14, 2022 15:29 Inactive
@andre-code andre-code temporarily deployed to renku-ci-ui-2137 November 15, 2022 09:02 Inactive
@andre-code andre-code temporarily deployed to renku-ci-ui-2137 November 15, 2022 12:38 Inactive
@lorenzo-cavazzi lorenzo-cavazzi force-pushed the 1743-show-jupyter-server-as-default branch from 52d6fbd to 89819a5 Compare November 18, 2022 15:16
@lorenzo-cavazzi lorenzo-cavazzi temporarily deployed to renku-ci-ui-2137 November 18, 2022 15:16 Inactive
@lorenzo-cavazzi lorenzo-cavazzi deployed to renku-ci-ui-2137 November 18, 2022 16:41 Active
@lorenzo-cavazzi lorenzo-cavazzi merged commit 7c415f3 into master Nov 18, 2022
@lorenzo-cavazzi lorenzo-cavazzi deleted the 1743-show-jupyter-server-as-default branch November 18, 2022 16:55
@RenkuBot
Copy link
Contributor

Tearing down the temporary RenkuLab deplyoment for this PR.

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.

show default jupyter-server tab in log view
3 participants