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

Update Origin UI to use API for the new Origin.Exports scheme #1042

Merged
merged 7 commits into from
Apr 8, 2024

Conversation

haoming29
Copy link
Contributor

Closes #1035

Since @jhiemstrawisc has a refactoring PR to support multiple S3 exports, which may add/change the scheme of Origin.Exports. Mark this PR as draft until Justin's PR gets merged in.

@haoming29 haoming29 added bug Something isn't working origin Issue relating to the origin component labels Apr 3, 2024
@haoming29 haoming29 modified the milestones: v7.8.0, v7.7.0 Apr 3, 2024
@haoming29 haoming29 marked this pull request as ready for review April 8, 2024 14:13
@haoming29 haoming29 requested a review from CannonLock April 8, 2024 14:33
Copy link
Contributor

@CannonLock CannonLock left a comment

Choose a reason for hiding this comment

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

LGTM. Ignored the resolved comment below, I patched that and forgot to delete the request.

web_ui/frontend/components/DataExportTable.tsx Outdated Show resolved Hide resolved
@haoming29 haoming29 merged commit 277c67d into PelicanPlatform:main Apr 8, 2024
19 checks passed
@haoming29 haoming29 deleted the data-export-API branch April 8, 2024 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working origin Issue relating to the origin component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Data Export table in origin UI
2 participants