Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add rate limit to nlweb template #772

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 29 commits into from
Sep 4, 2025
Merged

Add rate limit to nlweb template #772

merged 29 commits into from
Sep 4, 2025

Conversation

G4brym
Copy link
Member

@G4brym G4brym commented Aug 26, 2025 •
edited
Loading

Description

Add rate limit to nlweb template

Checklist

  • Template Metadata
    • template directory ends with -template
    • "cloudflare" section of package.json is populated
    • template preview image uploaded to Images
    • README is populated and uses <!-- dash-content-start --> and <!-- dash-content-end --> to designate the Dash readme preview
    • package.json contains a deploy command
    • package.json contains private: true and no version field

Example package.json

"private": true,
"cloudflare": {
  "label": "Worker + D1 Database",
  "products": [
    "Workers",
    "D1"
  ],
  "categories": [
    "storage"
  ],
  "docs_url": "https://developers.cloudflare.com/d1/",
  "preview_image_url": "https://imagedelivery.net/wSMYJvS3Xw-n339CbDyDIA/cb7cb0a9-6102-4822-633c-b76b7bb25900/public"
}

@G4brym G4brym marked this pull request as draft August 26, 2025 14:49
@G4brym G4brym marked this pull request as ready for review August 26, 2025 14:54
GregBrimble
GregBrimble reviewed Sep 4, 2025
View reviewed changes
nlweb-template/public/fp-chat-interface.js
}
}

markdownToHtml(markdown) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Markdown is a bit of a beast, and I'm pretty sure this won't actually capture the full spec, but I'm assuming this also isn't actually your file, so happy to leave it as it is if you are happy!

GregBrimble
GregBrimble reviewed Sep 4, 2025
View reviewed changes
nlweb-template/wrangler.jsonc
"ai": {
"binding": "AI"
},
"unsafe": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Just FYI this is about to stabilize: cloudflare/workers-sdk#10541

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 4, 2025

Preview link not generated: you must be on a branch, not on a fork.
Collaborators may enable previews for this pull request by attaching the allow preview label.
If you are already a collaborator, please create a branch rather than forking.

@GregBrimble GregBrimble merged commit 6a98d0f into cloudflare:main Sep 4, 2025
2 checks passed
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 4, 2025

Preview link not generated: you must be on a branch, not on a fork.
Collaborators may enable previews for this pull request by attaching the allow preview label.
If you are already a collaborator, please create a branch rather than forking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GregBrimble GregBrimble GregBrimble approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@G4brym @GregBrimble @danielgek