Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

First version of @dmno/cloudflare-platform #161

Merged
merged 6 commits into from
Dec 17, 2024
Merged

Conversation

philmillman
Copy link
Contributor

@philmillman philmillman commented Nov 15, 2024

  • types and schema for interacting with wrangler
  • types and schema for env vars injected by Cloudflare Pages
  • wrangler CLI wrapper to inject config

Also updated the #-api to use Cloudflare pages functions instead of Netlify, but need to work through deployment flow

  • also removed a few deps like axios and google sheets
  • added types for the email validator

Copy link

changeset-bot bot commented Nov 15, 2024

🦋 Changeset detected

Latest commit: ad544af

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
@dmno/cloudflare-platform Patch
@dmno/nextjs-integration Patch
@dmno/astro-integration Patch
@dmno/remix-integration Patch
dmno Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the maintainer Created by an official project maintainer label Nov 15, 2024
Copy link

netlify bot commented Nov 15, 2024

Deploy Preview for dmno ready!

Name Link
🔨 Latest commit ad544af
🔍 Latest deploy log https://app.netlify.com/sites/dmno/deploys/67613f9778fb6c0008672967
😎 Deploy Preview https://deploy-preview-161--dmno.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Nov 15, 2024

Deploy Preview for #-api ready!

Name Link
🔨 Latest commit 45bcb7e
🔍 Latest deploy log https://app.netlify.com/sites/#-api/deploys/67612f023d09050008ac981b
😎 Deploy Preview https://deploy-preview-161--#-api.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

cloudflare-workers-and-pages bot commented Nov 15, 2024

Deploying dmno with  Cloudflare Pages  Cloudflare Pages

Latest commit: ad544af
Status: ✅  Deploy successful!
Preview URL: https://554b101f.dmno.pages.dev
Branch Preview URL: https://move-to-cloudflare.dmno.pages.dev

View logs

Copy link

cloudflare-workers-and-pages bot commented Nov 15, 2024

Deploying #-api with  Cloudflare Pages  Cloudflare Pages

Latest commit: 45adc7e
Status: ✅  Deploy successful!
Preview URL: https://f86ff0bd.#-api.pages.dev
Branch Preview URL: https://move-to-cloudflare.#-api.pages.dev

View logs

@philmillman philmillman marked this pull request as ready for review November 18, 2024 14:33
@@ -1,4 +1,7 @@
functions-dist
functions/*.js
functions/*.map
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I realize this is a bit janky, but I didn't want built files to accidentally get committed

@@ -0,0 +1,4 @@
[build]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this might be unnecessary since it all lives in CF as well, we can revisit

@philmillman philmillman marked this pull request as draft November 20, 2024 21:58
@theoephraim theoephraim force-pushed the move-to-cloudflare branch 2 times, most recently from 162e8cc to d3a15bb Compare December 13, 2024 02:07
@theoephraim theoephraim marked this pull request as ready for review December 13, 2024 18:25
@theoephraim theoephraim force-pushed the move-to-cloudflare branch 2 times, most recently from 5e7268c to 45adc7e Compare December 17, 2024 06:57
Copy link

pkg-pr-new bot commented Dec 17, 2024

Open in Stackblitz

@dmno/configraph

npm i https://pkg.pr.new/@dmno/configraph@161

dmno

npm i https://pkg.pr.new/dmno@161

@dmno/astro-integration

npm i https://pkg.pr.new/@dmno/astro-integration@161

@dmno/nextjs-integration

npm i https://pkg.pr.new/@dmno/nextjs-integration@161

@dmno/remix-integration

npm i https://pkg.pr.new/@dmno/remix-integration@161

@dmno/vite-integration

npm i https://pkg.pr.new/@dmno/vite-integration@161

@dmno/cloudflare-platform

npm i https://pkg.pr.new/@dmno/cloudflare-platform@161

@dmno/1password-plugin

npm i https://pkg.pr.new/@dmno/1password-plugin@161

@dmno/bitwarden-plugin

npm i https://pkg.pr.new/@dmno/bitwarden-plugin@161

@dmno/encrypted-vault-plugin

npm i https://pkg.pr.new/@dmno/encrypted-vault-plugin@161

@dmno/infisical-plugin

npm i https://pkg.pr.new/@dmno/infisical-plugin@161

commit: ad544af

@theoephraim theoephraim changed the title move # api to cloudflare pages functions and refactor associated code First version of @dmno/cloudflare-platform Dec 17, 2024
@theoephraim theoephraim merged commit dc2425c into main Dec 17, 2024
9 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
maintainer Created by an official project maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants