Skip to content

[FEATURE] - PR6 : Web samples - #1201

Merged
canihavesomecoffee merged 1 commit into
CCExtractor:masterfrom
pulk17:web-samples
Sep 19, 2026
Merged

canihavesomecoffee merged 1 commit into
CCExtractor:masterfrom
pulk17:web-samples

Conversation

@pulk17

@pulk17 pulk17 commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

[FEATURE]

In raising this pull request, I confirm the following (please check boxes):

  • I have read and understood the contributors guide.
  • I have checked that another pull request for this purpose does not exist.
  • I have considered, and confirmed that this submission will be valuable to others.
  • I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
  • I give this submission freely, and claim no ownership to its content.

My familiarity with the project is as follows (check one):

  • I have never used the project.
  • I have used the project briefly.
  • I have used the project extensively, but have not contributed previously.
  • I am an active contributor to the project.

The sample library and uploads. Depends on #1200.

What's here
Samples with their parsed MediaInfo tree, tags, extra files, upload record, and which regression tests use them - plus per-platform status of the most recent run for that sample.

The upload side covers uploading a new sample, the queue of uploads awaiting processing, linking a queued upload to a sample or deleting it, and FTP credentials for large uploads.

Review notes
Media info is best-effort - a missing or unparseable XML shows as absent rather than failing the page, matching what the API does.

Uploads go to /samples/upload as plain multipart. Worth knowing that the nginx body limit in PR 9 is what makes this work at all for real files; today nginx would reject anything over 1 MB.

Deleting a sample warns that it takes its regression tests with it.

@pulk17 pulk17 changed the title Web samples [FEATURE] - PR6 : Web samples Sep 14, 2026
@pulk17 pulk17 mentioned this pull request Sep 14, 2026
9 tasks
Sample library with media info and per-sample test status, plus uploading a
new sample and working through the queue.
@sonarqubecloud

Copy link
Copy Markdown

@canihavesomecoffee
canihavesomecoffee merged commit 26475df into CCExtractor:master Sep 19, 2026
6 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