Change loadable.state to "loading", "success", and "error"

[hnordt]

loadable.state being “hasError” or “hasData” feels a bit off to me.

How about renaming it to something simpler like “error” or “success”? What do you think?

[dai-shi]

It's kind of too late. Migration isn't very easy and I don't know if it's worth it.

It's not that bad from the type definition. Maybe .state wasn't very appropriate.

jotai/src/vanilla/utils/loadable.ts

Lines 11 to 14 in d2a42f7

	export type Loadable<Value> =
	| { state: 'loading' }
	| { state: 'hasError'; error: unknown }
	| { state: 'hasData'; data: Awaited<Value> }

[hnordt]

What about adding a new prop “status” and deprecating “state”?

type Status = “loading” | “loaded” | “errored”

Or

type Status = “loading” | “success” | “error”

[dai-shi]

That's technically possible. I would like to get more feedback on this renaming.

[wojtekmaj]

With loadable being deprecated, this discussion could likely be closed.