feat: allow fetch override on client

[juliusmarminge]

quickly spun up the ability to allow fetch override to set headers. Ref: https://discord.com/channels/966627436387266600/1102510616326967306/1216240077605310576

Questions

• do we need full fetch override? or is headers enough? or both?
• currently only possible to set on the genUploader, generateReactHelpers, generateUploadButton etc etc. Maybe we should also have a way to allow this override when doing the call / on the components? Thinking if the token requires React context?

const { token } = useUser();
const { startUpload } = useUploadThing({
  headers: {
    Authorization: `Bearer ${token}`
  }
})

• is it even useful at all to have it on the generators? Maybe it should just be on individual calls

[changeset-bot]

⚠️ No Changeset found

Latest commit: c75b458

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
docs-uploadthing	✅ Ready (Inspect)	Visit Preview	Mar 10, 2024 10:04am

[github-actions]

A new canary is available for testing. You can install this latest build in your project with:

pnpm add @uploadthing/[email protected]
pnpm add @uploadthing/[email protected]
pnpm add @uploadthing/[email protected]
pnpm add @uploadthing/[email protected]
pnpm add @uploadthing/[email protected]
pnpm add [email protected]

[markflorkowski]

do we need full fetch override? or is headers enough? or both?

I can't really think of reasons why you would actually need a full fetch override, headers only is probably fine. IF other things are needed in the future we can reassess.

is it even useful at all to have it on the generators?

Probably not. I think having it on the hook/components is probably more flexible for the user.

[juliusmarminge]

Superseded by #705