Skip to content

Public sharing links for experiment notes and combined job chart#2044

Open
deep1401 wants to merge 7 commits into
mainfrom
add/public-share-links
Open

Public sharing links for experiment notes and combined job chart#2044
deep1401 wants to merge 7 commits into
mainfrom
add/public-share-links

Conversation

@deep1401
Copy link
Copy Markdown
Member

No description provided.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 11, 2026

Deploy Preview for transformerlab canceled.

Name Link
🔨 Latest commit 931bdad
🔍 Latest deploy log https://app.netlify.com/projects/transformerlab/deploys/6a076be5cda6830008602b69

@paragon-review
Copy link
Copy Markdown

Paragon Review Skipped

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

Please visit https://app.paragon.run to finish your review.

@sentry
Copy link
Copy Markdown

sentry Bot commented May 11, 2026

Codecov Report

❌ Patch coverage is 75.86207% with 35 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
api/transformerlab/routers/public_share.py 54.92% 29 Missing and 3 partials ⚠️
api/transformerlab/routers/experiment/share.py 90.62% 3 Missing ⚠️

📢 Thoughts on this report? Let us know!

deep1401 and others added 3 commits May 13, 2026 12:45
The early return for public share pages happened before useEffect calls,
violating react-hooks/rules-of-hooks and failing the react-doctor CI check.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.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.

1 participant