-
Notifications
You must be signed in to change notification settings - Fork 1.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
refactor(qrlToString): v2 SyncQRLInternal type #6338
Conversation
if (!backChannel) { | ||
backChannel = (globalThis as any)[QRL_RUNTIME_CHUNK] = new Map(); | ||
const isSync = isSyncQrl(value); | ||
if (!isSync) { |
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.
same as before but some two comments and avoiding the work for syncQRL
: ''); | ||
return qrlString; | ||
let qrlStringInline = `${chunk}#${symbol}`; | ||
if (Array.isArray(value.$captureRef$) && value.$captureRef$.length > 0) { |
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.
refactored to check for array and use for-loop for perf
$symbol$: typeof SYNC_QRL; | ||
} | ||
|
||
export type SyncQRLInternal = QRLInternal & SyncQRLSymbol; |
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.
create sync type
@Varixo I fixed the lint. idk how v2 tests work. is there v2 ci/cd and the errors are v1 things? if it's good can you merge? |
I think all good |
in v2.
add SyncQRLInternal