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: support downloading artifacts in the new emui #1720

Merged
merged 7 commits into from
Nov 8, 2023

Conversation

Dartoxian
Copy link
Contributor

Description:

This PR adds support for downloading artifacts as TGZ's from the new emui.

Additionally this PR:

  • updates the copy on the files table to File Artifacts
  • Makes files non clickable
  • Fixing the scrolling behaviour over monaco editors.

Is this change user facing?

YES

@Dartoxian Dartoxian added the DO NOT MERGE DO NOT MERGE THIS PR label Nov 8, 2023
@adschwartz adschwartz marked this pull request as ready for review November 8, 2023 19:45
## Description:
This PR implements all burdown feedback for:
* Enclave list view (make clickable cells work properly, make headers
highlightable when sortable, image names clickable, make non clickable
styles work with correct styling)
* Enclave overview (remove source tab, fix cell interactivity in table
for ports, merge public private ports in popover, drop ports timeout)
* Enclave logs (link to existing issue rather than suggesting make a new
issue)
* Service view (fix monaco scrolling, remove delete/edit buttons, merge
public/private ports)
* Enclave configuration (copy updates, but not json field update).

## Is this change user facing?
YES

## References (if applicable):
*
https://www.notion.so/kurtosistech/FE-Devconnect-Work-breakdown-af548547e0ec4a4996f75205a4fcbd60#b9f4e5a71aad4347827473e72bc888b2
@adschwartz adschwartz removed the DO NOT MERGE DO NOT MERGE THIS PR label Nov 8, 2023
@adschwartz adschwartz added this pull request to the merge queue Nov 8, 2023
Merged via the queue into main with commit fbfeaa3 Nov 8, 2023
34 checks passed
@adschwartz adschwartz deleted the feat/download-files branch November 8, 2023 20:51
github-merge-queue bot pushed a commit that referenced this pull request Nov 8, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.85.8](0.85.7...0.85.8)
(2023-11-08)


### Features

* support downloading artifacts in the new emui
([#1720](#1720))
([fbfeaa3](fbfeaa3))


### Bug Fixes

* stop all services first to update the service status in the service
registration during a `kurtosis enclave stop` execution
([#1712](#1712))
([3d1e142](3d1e142)),
closes [#1711](#1711)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: kurtosisbot <kurtosisbot@users.noreply.github.com>
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.

3 participants