-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
reflect params in the debugger #148
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @shakyShane and the rest of your teammates on Graphite |
@@ -21,5 +21,5 @@ | |||
"jsxFactory": "h", | |||
"jsxFragmentFactory": "Fragment" | |||
}, | |||
"include": ["types.ts", "schema", "e2e", "scripts", "shared/js", "integration-tests", "guides"] | |||
"include": ["types.ts", "schema", "e2e", "scripts", "shared/js", "integration-tests", "guides", "debugger"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the debugger folder was previously missed (by mistake)
let reflectList = ['screen'] | ||
let reflectParams = new URLSearchParams(Object.entries({ state: initialState })) | ||
for (let [key, value] of searchParams) { | ||
if (reflectList.includes(key)) { | ||
reflectParams.set(key, value) | ||
} | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This represents a set of keys that might be set on the outer debugger iframe. Anything in reflectList
should be forwarded to each iframe instance
a861e11
to
efffc8b
Compare
140d6a0
to
1f69e7a
Compare
1f69e7a
to
40c07b8
Compare
40c07b8
to
c1dbb19
Compare
https://app.asana.com/0/0/1207195637811329/f
TL;DR
Update debugger to reflect url parameters and fix initial state loading. This will be needed in the next PR to preview/debug another new screen.
What changed?
How to test?
npm run preview
screen-breakageForm
, orscreen-toggleReport
Why make this change?
This change ensures the debugger reflects correct URL parameters - to mimic what the native applications do. Without this change, you could not load the dashboard in this state, making debugging more difficult