-
Notifications
You must be signed in to change notification settings - Fork 2k
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: insertBefore加入script逻辑 #592
Conversation
); | ||
|
||
const dynamicScriptCommentElement = document.createComment(`dynamic script ${src} replaced by qiankun`); | ||
return rawAppendChild.call(appWrapperGetter(), dynamicScriptCommentElement) as T; |
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.
这里应该是 rawHeadInsertBefore
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.
好的, 已改
element.onerror, | ||
); | ||
const dynamicInlineScriptCommentElement = document.createComment('dynamic inline script replaced by qiankun'); | ||
return rawHeadInsertBefore.call(appWrapperGetter(), dynamicInlineScriptCommentElement, refChild) as T; |
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.
Checklist
npm test
passesDescription of change
resolve #590
dynamicHeadAppend.ts中insertBefore加入script处理逻辑