Skip to content

Fix team gallery tasks and team interactive tasks#1566

Merged
deep1401 merged 14 commits intomainfrom
fix/team-gallery-tasks
Mar 18, 2026
Merged

Fix team gallery tasks and team interactive tasks#1566
deep1401 merged 14 commits intomainfrom
fix/team-gallery-tasks

Conversation

@deep1401
Copy link
Copy Markdown
Member

  • Also adds view files for team tasks
  • Also adds a file viewer to show the task.yaml in the team gallery
  • Temporarily hides the tab for team interactive tasks

@paragon-review
Copy link
Copy Markdown

Paragon Review Skipped

Hi @deep1401! Your Polarity credit balance is insufficient to complete this review.

Please visit https://home.polarity.cc to add more credits and continue using Paragon reviews.

@sentry
Copy link
Copy Markdown

sentry bot commented Mar 17, 2026

Codecov Report

❌ Patch coverage is 34.36123% with 149 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
api/transformerlab/routers/experiment/task.py 33.77% 133 Missing and 16 partials ⚠️

📢 Thoughts on this report? Let us know!

Copy link
Copy Markdown
Member

@dadmobile dadmobile left a comment

Choose a reason for hiding this comment

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

Mostly just a few questions inline. My only broader question is wondering if any of this code is duplicated from regular import and could be refactored easily? I can't tell which maybe suggests it would not be easy...in which case we could make a followup if it is worthwhile.

Comment thread api/transformerlab/routers/experiment/task.py Outdated
Comment thread src/renderer/components/Experiment/Tasks/FileBrowserModal.tsx
Comment thread src/renderer/lib/api-client/endpoints.ts
Comment thread api/transformerlab/routers/experiment/task.py Outdated
@deep1401 deep1401 changed the title Fix team gallery tasks Fix team gallery tasks and team interactive tasks Mar 17, 2026
Copy link
Copy Markdown
Member Author

@deep1401 deep1401 left a comment

Choose a reason for hiding this comment

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

Made a push to also have export on interactive tasks in the same way

@deep1401 deep1401 requested a review from dadmobile March 17, 2026 21:08
Copy link
Copy Markdown
Member

@dadmobile dadmobile left a comment

Choose a reason for hiding this comment

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

Sorry for unapproving. Just seeing codecov and I think it's probably worth adding a simple test for this?

Copy link
Copy Markdown
Member Author

@deep1401 deep1401 left a comment

Choose a reason for hiding this comment

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

Sorry for unapproving. Just seeing codecov and I think it's probably worth adding a simple test for this?

Added a team task import export test now

@deep1401 deep1401 requested a review from dadmobile March 18, 2026 13:46
@deep1401 deep1401 merged commit cdb904b into main Mar 18, 2026
9 checks passed
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.

2 participants