Skip to content

[core/models/design-request] Add snapfish default style#34

Merged
54nd10 merged 2 commits intomainfrom
snapfish-default-style
May 14, 2025
Merged

[core/models/design-request] Add snapfish default style#34
54nd10 merged 2 commits intomainfrom
snapfish-default-style

Conversation

@jemetri
Copy link
Contributor

@jemetri jemetri commented May 13, 2025

Shutterfly does not have any SKUs

So if no style is provided but an sku is provided -> Default to snapfish style
If no style is provided and no sku is provided -> Default to modern-black-sfly

@github-actions
Copy link

github-actions bot commented May 13, 2025

Coverage Summary

Status Category Percentage Covered / Total
🔵 Lines 100% 340 / 340
🔵 Statements 100% 340 / 340
🔵 Functions 100% 75 / 75
🔵 Branches 100% 128 / 128
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
packages/cli/src/commands/books.ts 100% 100% 100% 100%
packages/cli/src/commands/config.ts 100% 100% 100% 100%
packages/cli/src/commands/design-options.ts 100% 100% 100% 100%
packages/cli/src/commands/images.ts 100% 100% 100% 100%
packages/cli/src/commands/mb-client.ts 100% 100% 100% 100%
packages/cli/src/commands/spreads.ts 100% 100% 100% 100%
packages/cli/src/commands/storyboard-items.ts 100% 100% 100% 100%
packages/cli/src/models/config.ts 100% 100% 100% 100%
packages/cli/src/utils/toolbox.ts 100% 100% 100% 100%

@54nd10 54nd10 changed the title add snapfish default style [core/models/design-request] Add snapfish default style May 14, 2025
@54nd10 54nd10 merged commit 35d382a into main May 14, 2025
3 checks passed
@54nd10 54nd10 deleted the snapfish-default-style branch May 14, 2025 09:08
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