lazyObservable.current is missing `| undefined` in type signature
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Newbie friendliness
- 45/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Stale
- Tech stack
- typescript
- Domain
- tooling
Research direction
Start in src/lazy-observable.ts around line 45 and read the linked code example showing current() compared with undefined. Check the TypeScript return type against the unresolved-promise behavior; done means the public signature accurately permits the value demonstrated by the example.
Written by the indexing model from the issue text.
Description
Given that we're fine to omit the initial value fallback, current() can return undefined while the promise is unresolved. The code snippet example even compares current() === undefined.
- Dominant language
- TypeScript
- Stars
- 1.2k
- Forks
- 130
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from mobxjs/mobx-utils
-
Difficulty 3/5 1-2 days Newbie friendliness 38/100
mobxjs/mobx-utils#338 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
mobxjs/mobx-utils#330 · 2 comments ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 45/100
mobxjs/mobx-utils#329 · 1 reaction ·
-
Difficulty 3/5 1-2 days Newbie friendliness 52/100
mobxjs/mobx-utils#327 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
mobxjs/mobx-utils#324 · 2 reactions ·
All issues in mobxjs/mobx-utils
Similar issues
-
clawsweeper:fix-shape-clear clawsweeper:queueable-fix clawsweeper:source-repro impact:ux-friction issue-rating: 🦞 diamond lobster no-stale P3
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Difficulty 1/5 Under an hour Newbie friendliness 76/100
-
code-quality refactoring
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
github/gh-aw-firewall#8816 ·
-
integration:quickjs org:external priority:backlog topic:code-interpreter topic:middleware type:feature
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
langchain-ai/deepagents#6450 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 88/100
vercel/react-tweet#225 ·