fix: pass auth profiles to vite - #14588
Conversation
🦋 Changeset detectedLatest commit: 3ede3c3 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
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 |
|
Review posted successfully to PR #14588. I reviewed the PR, which adds an import-time
|
@cloudflare/autoconfig
create-cloudflare
@cloudflare/deploy-helpers
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-shared
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-pool-workers
@cloudflare/workers-auth
@cloudflare/workers-editor-shared
@cloudflare/workers-utils
wrangler
commit: |
bb98bf0 to
8e9a085
Compare
|
✅ All changesets look good |
8e9a085 to
7400be0
Compare
7400be0 to
953cb57
Compare
|
Codeowners approval required for this PR:
Show detailed file reviewers |
f6092b6 to
bf8a21d
Compare
jamesopstad
left a comment
There was a problem hiding this comment.
Is there some way we can test this so it doesn't regress or is it too difficult?
workers-devprod
left a comment
There was a problem hiding this comment.
Codeowners reviews satisfied
55e252e to
3ede3c3
Compare
Fixes #14580
wrangler logout; wrangler auth profile create/activate 'test'; vite devin a directory with remote bindings. no oauth flow triggered, remote bindings used the 'test' profile. then deleted profile, and it required re-authing.A picture of a cute animal (not mandatory, but encouraged)