Skip to content

Conversation

@tsnobip
Copy link
Collaborator

@tsnobip tsnobip commented Jul 10, 2025

No description provided.

@tsnobip tsnobip force-pushed the refetch-variable-custom-scalars branch 2 times, most recently from 640ba55 to 0b9ec14 Compare July 10, 2025 14:20
@tsnobip tsnobip force-pushed the refetch-variable-custom-scalars branch 3 times, most recently from e6580f6 to 592be0b Compare July 10, 2025 14:29
@tsnobip tsnobip force-pushed the refetch-variable-custom-scalars branch from 592be0b to aa0e5da Compare July 10, 2025 16:47
@tsnobip tsnobip changed the title add test for refetch variable with None custom scalar fix conversion of refetch variable with None custom scalar Jul 10, 2025
Comment on lines +82 to +87
if (currentObj[key] && currentObj[key].BS_PRIVATE_NESTED_SOME_NONE >= 0) {
newObj = getNewObj(newObj, currentObj);
newObj[key] = currentObj[key];
continue;
}

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here's the fix

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fantastic! Great find. Would you add a comment in the code about what this is?

@zth
Copy link
Owner

zth commented Jul 10, 2025

Amazing! Would you add a changelog as well?

@tsnobip
Copy link
Collaborator Author

tsnobip commented Jul 10, 2025

Woops I forgot about that!

@tsnobip
Copy link
Collaborator Author

tsnobip commented Jul 11, 2025

@zth done!

@tsnobip tsnobip requested a review from zth July 11, 2025 07:23
@zth zth merged commit 5a8532a into zth:master Jul 11, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants