emberjs / emberjs/ember-inspector

Unify ember_debug's usage of `owner` vs `_application`

Open
#1,199 2 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
JavaScript
Stars
969
Forks
292
PR merge metrics
No merged PRs in 30d

Description

Currently, `EmberDebug` uses `owner` in our own tests, but in production we _always_ use `_application.__deprecatedInstance__`. This split is kinda bad to begin with but it also prevents issues with using the ember-inspector while running an Apps own tests.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating EmberDebug and its test and production references to owner and _application.__deprecatedInstance__; compare how each path obtains the application. Confirm that the chosen access pattern works both during an app's own tests and in normal production use, then run the relevant EmberDebug tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
devtools
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.