Skip to content

chore: enable Cloudflare custom domains in wrangler config - #185

Merged
ayoub3bidi merged 1 commit into
developfrom
chore/wrangler-update
Jun 23, 2026
Merged

chore: enable Cloudflare custom domains in wrangler config#185
ayoub3bidi merged 1 commit into
developfrom
chore/wrangler-update

Conversation

@ayoub3bidi

@ayoub3bidi ayoub3bidi commented Jun 23, 2026

Copy link
Copy Markdown
Owner

Contribution workflow

  • Base branch is develop: This PR targets develop, not main.
  • Guidelines and docs: Read CONTRIBUTING.md and relevant docs.
  • This template: PR template structure kept; sections below filled in.

Description

Enables custom-domain routes in wrangler.jsonc for the Netlify → Cloudflare migration (phases 3–4). workers.dev smoke tests are already done; this wires the Workers to the real hostnames so deploys can attach dev.bayanflow.com and bayanflow.com.

Note: Merging to develop only updates the staging worker (dev.bayanflow.com). Production routes apply after this reaches main via a follow-up merge.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • 🎨 Style/UI improvement
  • ⚡ Performance improvement
  • 🧪 Test addition or improvement
  • 🔧 Chore (maintenance, dependencies, etc.)

Related Issues

Fixes #

Changes Made

  • Uncomment / add routes on env.stagingdev.bayanflow.com
  • Uncomment / add routes on env.productionbayanflow.com, www.bayanflow.com
  • Update wrangler.jsonc comments to reflect phases 1–2 complete

Algorithm Details (if applicable)

N/A

Testing

  • workers.dev URLs smoke-tested before this change
  • After merge: Deploy (Cloudflare Workers) green on develop
  • After merge: confirm dev.bayanflow.com appears on bayan-flow-staging → Domains tab
  • DNS cutover for dev.bayanflow.com (manual, post-deploy)
  • Production domains after merge to main + deploy + DNS

Test Results

Config-only change; validation is deploy + live URL check.

Screenshots/GIFs

N/A

Code Quality

  • Minimal diff; no application code touched
  • Matches existing wrangler.jsonc environment layout

Performance Impact

  • No performance impact

Accessibility

  • N/A

Breaking Changes

  • None in repo code. DNS cutover to Cloudflare is a manual ops step after deploy; Netlify remains rollback until domains are verified (~2 weeks).

Checklist

  • Contribution workflow checklist completed
  • CI green on PR
  • Staging deploy succeeds with custom domain attached
  • Follow-up: merge to main for production routes + bayanflow.com DNS

Additional Notes

After this PR merges to develop

  1. Confirm Deploy (Cloudflare Workers) succeeds
  2. bayan-flow-staging → Domains → dev.bayanflow.com active
  3. Update DNS (remove Netlify records if still pointing there)
  4. Smoke https://dev.bayanflow.com

Production (bayanflow.com)

Requires the same wrangler.jsonc on main (merge developmain when staging is verified).

Rollback

Point DNS back to Netlify; Netlify projects kept intentionally.


Reviewer Guidelines:

  • Verify route patterns match live hostnames only
  • No app or registry changes expected

Summary by CodeRabbit

  • Chores
    • Updated deployment configuration with phased rollout descriptions, including domain repointing guidance and rollback period information.
    • Configured custom domain routes for staging (dev.bayanflow.com) and production (bayanflow.com, www.bayanflow.com) environments.

@ayoub3bidi ayoub3bidi self-assigned this Jun 23, 2026
@netlify

netlify Bot commented Jun 23, 2026

Copy link
Copy Markdown

Deploy Preview for dev-bayanflow ready!

Name Link
🔨 Latest commit 84d9d9c
🔍 Latest deploy log https://app.netlify.com/projects/dev-bayanflow/deploys/6a3af62108331900089df1eb
😎 Deploy Preview https://deploy-preview-185--dev-bayanflow.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@github-actions github-actions Bot added the ci Workflows label Jun 23, 2026
@coderabbitai

coderabbitai Bot commented Jun 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 5dc4c767-50f7-462f-8789-e43d8fc39334

📥 Commits

Reviewing files that changed from the base of the PR and between 2b0e649 and 84d9d9c.

📒 Files selected for processing (1)
  • wrangler.jsonc

📝 Walkthrough

Walkthrough

Updates wrangler.jsonc with explicit custom domain routes for staging (dev.bayanflow.com) and production (bayanflow.com, www.bayanflow.com), which were previously commented out. The top-of-file comment block is replaced with a phased rollout description including domain repointing steps and a ~2-week rollback window.

Changes

Wrangler Deployment Configuration

Layer / File(s) Summary
Phase comments and custom domain route activation
wrangler.jsonc
Replaces the generic migration comment with a numbered phased rollout/rollback description. Adds dev.bayanflow.com as an explicit custom domain route for staging, and bayanflow.com plus www.bayanflow.com as explicit custom domain routes for production.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • ayoub3bidi/bayan-flow#183: Directly overlaps at the same wrangler.jsonc file, modifying Cloudflare environment and custom-domain route configuration for staging and production.

Suggested labels

ci

Poem

🐇 Hop, hop, domains now live and bright,
bayanflow.com pointed just right!
Staging whispers dev with glee,
Production roars for all to see.
A two-week window, safe and neat —
The rabbit's deploy is now complete! 🌸

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'chore: enable Cloudflare custom domains in wrangler config' accurately describes the main change—uncommenting and enabling custom domain routes in wrangler.jsonc for staging and production environments.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/wrangler-update

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

Cloudflare Workers preview for bayan-flow-staging ready!

Name Link
🔨 Latest commit 84d9d9c
🔍 Latest deploy log https://github.com/ayoub3bidi/bayan-flow/actions/runs/28057299660
😎 Deploy Preview https://pr-185-bayan-flow-staging.ayoub3bidi.workers.dev
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

Preview alias pr-185 on the staging worker. Updates automatically with new commits.

@codecov

codecov Bot commented Jun 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@ayoub3bidi
ayoub3bidi merged commit 6eca5d9 into develop Jun 23, 2026
15 checks passed
@ayoub3bidi
ayoub3bidi deleted the chore/wrangler-update branch June 23, 2026 21:12
@ayoub3bidi ayoub3bidi mentioned this pull request Jul 16, 2026
35 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant