switch to component-hosted http#22
Open
ianmacartney wants to merge 29 commits into
Open
Conversation
Merged
commit: |
ianmacartney
commented
May 13, 2026
| exmple/dist/ | ||
|
|
||
| # Note: dist/ is NOT ignored - it's committed for GitHub installs | ||
| # (npm install github:get-convex/static-hosting#main) |
Member
Author
There was a problem hiding this comment.
instead using prepare: "npm run build" in package scripts
sethconvex
approved these changes
May 13, 2026
ianmacartney
changed the base branch from
ian/use-convex-eslint
to
graphite-base/22
Member
Author
|
My main hesitation here is how to make it (more) backwards-compatible -
open to ideas on how to steer agents towards a migration guide, etc. -
maybe the changelog is enough?
…On Wed, May 13, 2026 at 9:52 AM sethconvex ***@***.***> wrote:
***@***.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
<#22 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACZQWYKTISDRW4JFAVGQ7L42SR6HAVCNFSM6AAAAACY4FSEQSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHM2DEOBTG4ZTQMRVGY>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
ianmacartney
force-pushed
the
ian/http-revamp
branch
from
c9cf38d to
423136f
Compare
ianmacartney
force-pushed
the
ian/http-revamp
branch
from
14be793 to
c35b8a3
Compare
ianmacartney
force-pushed
the
ian/http-revamp
branch
from
1942599 to
3b261a9
Compare
Member
Author
Member
|
@ianmacartney yep checked it here on windows and those fixees are still in and seems to work. Thanks. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Use the component's file storage and http serving, removing ~all app config
(unless you use UpdateBanner, which now reactively tells you where to put the export)
This also simplifies the CLI, which now only needs to add an entry to convex.config.ts and a deploy package script.
It also swaps the "check in dist" strategy for a "prepare" script which runs a build when being installed - so you can still install from github (if you want - but is that necessary now that we have the pkg-pr-new thing?)