Skip to content

feat: add prepr cms image provider integration - #823

Merged
danielroe merged 21 commits into
nuxt:mainfrom
FranciscoKloganB:feat/add-preprio-provider
Jun 12, 2023
Merged

feat: add prepr cms image provider integration#823
danielroe merged 21 commits into
nuxt:mainfrom
FranciscoKloganB:feat/add-preprio-provider

Conversation

@FranciscoKloganB

@FranciscoKloganB FranciscoKloganB commented May 13, 2023

Copy link
Copy Markdown
Contributor

Description

This PR adds an integration with Prepr Headless CMS, docs can be found at Prepr Docs.

Motivation and Context

While the image transformation options provided by this CMS are still few, their Support team informs me, that they are due to receive updates very soon. I also happen to need to use this provider on my personal project and I much prefer to share the provider with the community then have it all for myself. ❤️

Changes

  • Sorts BuiltInProviders array alphabetically, no removals were made, only prepr key was added.
  • Add src/runtime/prepr folder containing the necessary code, to getImage.
  • Add prepr documentation to docs/content
  • Add necessary configurations for playground

@ghost

ghost commented May 13, 2023

Copy link
Copy Markdown

Live Preview ready!

Name Edit Preview Latest Commit
Image Edit on Studio ↗︎ View Live Preview 9523440

@netlify

netlify Bot commented May 13, 2023

Copy link
Copy Markdown

👷 Deploy request for nuxt-image pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 9523440

@danielroe danielroe changed the title feat(providers): add prepr cms image provider integration, documents and playground feat: add prepr cms image provider integration May 28, 2023
@danielroe danielroe added enhancement New feature or request provider labels May 28, 2023
@danielroe

Copy link
Copy Markdown
Member

Would you be able to add snapshots/expected values for prepr? 🙏

@FranciscoKloganB

Copy link
Copy Markdown
Contributor Author

@danielroe Hey, I think anything is possible. Can you just clarity what you are looking for? Thanks

@danielroe

Copy link
Copy Markdown
Member

Merge in the latest changes in the main branch, and get the tests passing. (This requires adding prepr entries to the unit tests, just like the other providers.)

@FranciscoKloganB

Copy link
Copy Markdown
Contributor Author

Makes sense. Will do. :)

@FranciscoKloganB

Copy link
Copy Markdown
Contributor Author

@danielroe I think that's it :)

Comment thread docs/content/5.providers/prepr.md Outdated
Comment thread test/unit/providers.test.ts Outdated
Comment thread test/e2e/__snapshots__/ssr.test.ts.snap Outdated
Comment thread test/e2e/__snapshots__/no-ssr.test.ts.snap Outdated
Comment thread src/runtime/providers/prepr/getImage.ts Outdated
Comment thread src/runtime/providers/prepr/getImage.ts Outdated
FranciscoKloganB and others added 2 commits June 7, 2023 21:51
Co-authored-by: Daniel Roe <daniel@roe.dev>
Co-authored-by: Daniel Roe <daniel@roe.dev>
Comment thread docs/content/5.providers/prepr.md Outdated
Comment thread docs/content/5.providers/prepr.md Outdated

@danielroe danielroe left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think probably we could inline all the small helper files in the provider directory in a single file, but I'm not really sure it makes much difference.

Thank you for this - it'll be a great addition to the module!

@danielroe
danielroe requested a review from pi0 June 8, 2023 09:52
@danielroe danielroe changed the title feat: add prepr cms image provider integration feat: add prepr cms image provider integration Jun 12, 2023
@danielroe
danielroe merged commit 6b8ab9c into nuxt:main Jun 12, 2023
@danielroe danielroe mentioned this pull request Jun 20, 2023
@github-actions github-actions Bot mentioned this pull request Jul 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request provider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants