Skip to content
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

chore: remove unused ai binding #308

Merged
merged 1 commit into from
Oct 9, 2024
Merged

chore: remove unused ai binding #308

merged 1 commit into from
Oct 9, 2024

Conversation

RihanArfan
Copy link
Contributor

Closes DEV-363

@pkg-pr-new pkg.pr.new
Copy link

pkg-pr-new bot commented Oct 7, 2024

Open in Stackblitz

pnpm add https://pkg.pr.new/nuxt-hub/core/@nuxthub/core@308

commit: 40d8282

@atinux atinux merged commit 54db127 into nuxt-hub:main Oct 9, 2024
4 of 5 checks passed
@cernymatej
Copy link

Hi there! I think this might have caused an issue where using hubAI() during local development now throws an error saying: Missing Cloudflare AI binding (AI).

The generated file .data/hub/wrangler.toml no longer includes the AI binding. Why was this change made? Why is it considered unused?

@danielroe danielroe mentioned this pull request Oct 16, 2024
Closed
@RihanArfan RihanArfan deleted the fix/unused-ai-binding branch October 30, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@RihanArfan @cernymatej @atinux