Observable property name is not show in diff popover using mobx v5
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 42/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- javascript, react
Research direction
Start in src/frontend/TabChanges/LObjDiff.jsx at the referenced use of this.props.path, then compare it with the MobX v3 and v5 behavior shown in the linked example and upgrade commit. Trace where the path is defined and confirm that the diff popover displays the observable property name rather than the MobX class name and number.
Written by the indexing model from the issue text.
Description
When using mobx v3, the popover that shows object diff shows the property name (isLoading) correctly as below:
When upgrading to mobx v5 is show the class name followed by a number (ArticlesStore@9) instead of the property name
Is this a know bug in devtools?
Or is a problem with mobx v5?
What code i can look for it?
The Popover code is here but i could not find where this.props.path value is defined
This is the original code where i found this behavior and the diff upgrading mobx to v5
- Dominant language
- JavaScript
- Stars
- 503
- Forks
- 34
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
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-devtools
-
Difficulty 2/5 Half a day Newbie friendliness 68/100
mobxjs/mobx-devtools#158 · 1 reaction ·
-
Difficulty 1/5 Under an hour Newbie friendliness 68/100
mobxjs/mobx-devtools#62 · 1 comment ·
-
Difficulty 4/5 3-5 days Newbie friendliness 28/100
mobxjs/mobx-devtools#114 · 1 reaction ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 35/100
mobxjs/mobx-devtools#111 · 1 comment · 8 reactions ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 48/100
mobxjs/mobx-devtools#110 · 3 comments · 10 reactions ·
All issues in mobxjs/mobx-devtools
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 88/100
jspreadsheet/ce#1809 ·
-
documentation
Difficulty 1/5 Under an hour Newbie friendliness 91/100
githubnext/gh-aw-workshop#4458 ·
Maintainers usually reply within 1 day
-
Add: CartoonitoOpencheck:failed feeds:add
Difficulty 2/5 1-3 hours Newbie friendliness 63/100
iptv-org/database#37390 · 1 comment ·
Maintainers usually reply within 9 days
-
bug: directory index route root priority is overwritten when wildcard is falsePossibly taken @TalhaHunter101 claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
fastify/fastify-static#617 ·
-
bug confirmed css v6
Difficulty 2/5 1-3 hours Newbie friendliness 66/100
Maintainers usually reply within 1 day