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

Import bdbag + fix log levels #225

Open
wants to merge 28 commits into
base: master
Choose a base branch
from
Open

Import bdbag + fix log levels #225

wants to merge 28 commits into from

Conversation

bodom0015
Copy link
Member

@bodom0015 bodom0015 commented Sep 28, 2021

Problem

Fixes #221
Fixes #214

Approach

How to Test

Prerequisites: at least one Tale created

  1. Checkout this branch locally, rebuild the dashboard
  2. Login to the WholeTale Dashboard
  3. Navigate to Run > Files > External Data for a Tale that you own
  4. Expand the dropdown at the top-right
    • You should see a new option to Import BDBag (text TBD)
  5. Click Import BDBag and choose a file to upload
    • You should see a notification appear indicating the the BDBag is being uploaded
  6. Wait for the upload/import to complete
  7. Expand the dropdown at the top-right once more and choose "WT Data Catalog"
    • You should see the Select Data dialog appear
    • You should see the Bag you just imported is now listed here as well

@bodom0015 bodom0015 requested a review from Xarthisius November 8, 2021 21:55
@bodom0015
Copy link
Member Author

Updated this PR to hide the Import BDBag option, as requested... rest of the PR can be merged so that the changes to the API client are preserved 👍

@bodom0015 bodom0015 changed the base branch from recorded-runs-redux to master May 23, 2022 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant