Skip to content

Add step to photo processing#3412

Merged
ildyria merged 2 commits intoadd-extract-color-jobfrom
add-step-to-processing
Jun 7, 2025
Merged

Add step to photo processing#3412
ildyria merged 2 commits intoadd-extract-color-jobfrom
add-step-to-processing

Conversation

@ildyria
Copy link
Copy Markdown
Member

@ildyria ildyria commented Jun 5, 2025

This pull request introduces a new feature to extract color palettes from images during photo creation. The changes include adding a new pipeline class, integrating it into existing workflows, and ensuring it respects configuration settings.

New Feature: Color Palette Extraction

  • Integration into photo creation workflows:

    • Added the Shared\ExtractColourPalette pipeline to the handleStandalone method in app/Actions/Photo/Create.php to extract color palettes during standalone photo creation.
    • Added the Shared\ExtractColourPalette pipeline to the handlePhotoLivePartner method in app/Actions/Photo/Create.php to extract color palettes during partner photo creation.
  • New pipeline class:

    • Introduced ExtractColourPalette in app/Actions/Photo/Pipes/Shared/ExtractColourPalette.php, which handles color extraction using ExtractColoursJob. The pipeline respects configuration flags for enabling color extraction and using job queues.

@ildyria ildyria requested a review from a team as a code owner June 5, 2025 16:16
@ildyria ildyria force-pushed the add-step-to-processing branch from f89d425 to b9274e4 Compare June 5, 2025 17:23
@ildyria ildyria force-pushed the add-extract-color-job branch 2 times, most recently from 0203bfe to 780ff10 Compare June 5, 2025 19:08
@ildyria ildyria force-pushed the add-step-to-processing branch from b9274e4 to 4349ca2 Compare June 5, 2025 19:31
@ildyria ildyria force-pushed the add-extract-color-job branch from 780ff10 to fca1e88 Compare June 5, 2025 20:08
@ildyria ildyria force-pushed the add-step-to-processing branch 2 times, most recently from 7f568b6 to ed38278 Compare June 6, 2025 10:10
@ildyria ildyria added the Review: easy Easy review expected: probably just need a quick to go through. label Jun 6, 2025
@ildyria ildyria force-pushed the add-step-to-processing branch from ed38278 to b85d9ff Compare June 6, 2025 12:17
@ildyria ildyria force-pushed the add-extract-color-job branch from fca1e88 to 3b9b3bf Compare June 6, 2025 15:09
@ildyria ildyria force-pushed the add-step-to-processing branch from b85d9ff to 0129299 Compare June 6, 2025 15:09
@ildyria ildyria force-pushed the add-extract-color-job branch 2 times, most recently from fcbfd6c to 0073bb4 Compare June 6, 2025 15:32
@ildyria ildyria force-pushed the add-step-to-processing branch from 0129299 to 0162ed7 Compare June 6, 2025 15:39
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 7, 2025

Codecov Report

Attention: Patch coverage is 52.63158% with 27 lines in your changes missing coverage. Please review.

Project coverage is 86.66%. Comparing base (0073bb4) to head (f4a956a).
Report is 1 commits behind head on add-extract-color-job.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ildyria ildyria merged commit 75ce78d into add-extract-color-job Jun 7, 2025
33 checks passed
@ildyria ildyria deleted the add-step-to-processing branch June 7, 2025 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review: easy Easy review expected: probably just need a quick to go through.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants