-
Notifications
You must be signed in to change notification settings - Fork 2.9k
feat: DIA-1924: "Import sample data" ability in the Data Import page #7206
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
Conversation
✅ Deploy Preview for label-studio-docs-new-theme canceled.
|
✅ Deploy Preview for heartex-docs canceled.
|
/fm sync |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #7206 +/- ##
========================================
Coverage 77.17% 77.18%
========================================
Files 187 187
Lines 14499 14504 +5
========================================
+ Hits 11190 11195 +5
Misses 3309 3309
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
7958c28
to
866cc83
Compare
…ast messages in favour of modals with increased Sentry reporting (#7167) Co-authored-by: robot-ci-heartex <[email protected]>
… results in ERROR 500 on develop (#7203)
… results in ERROR 500 on develop (#7207)
Co-authored-by: robot-ci-heartex <[email protected]> Co-authored-by: mcanu <[email protected]>
…7209) Co-authored-by: robot-ci-heartex <[email protected]>
web/apps/labelstudio/src/pages/CreateProject/Import/ImportModal.jsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
small observation on the package.json, and I agree with the other feedback that we should look to make the hooks in the same way to align with the existing.
2544ac0
to
b5d8f7a
Compare
Sample 1 (images) is not actually importing - looking into it. Sample 2 works. edit: fixed |
Co-authored-by: bmartel <[email protected]>
@yyassi-heartex @bmartel @nass600 please, take another look |
[x] add sample data import component
[x] add configurable manifest file with sample data
[x] create default manifest file with data hosted on our s3