Skip to content

Conversation

OskarDamkjaer
Copy link
Contributor

@OskarDamkjaer OskarDamkjaer requested a review from jharris4 June 14, 2021 14:04
@@ -72,7 +72,7 @@ function CypherScriptFrame({
data-testid="multi-statement-list-title"
{...titleProps}
>
<PointerFrameCommand selectedDb={frames[id].cmd}>
<PointerFrameCommand selectedDb={frames[id].useDb || ''}>
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice find, is the precedent to use empty string vs null/undefined when there is no db?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed this in another commit

@@ -223,7 +223,15 @@ Third-party licenses
│ ├─ long@4.0.0
│ │ ├─ URL: https://github.com/dcodeIO/long.js.git
│ │ └─ VendorName: Daniel Wirtz
│ ├─ neo4j-driver@4.2.2
│ ├─ neo4j-driver-bolt-connection@4.3.1
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice, licenses update seems to have been missed with the last driver upgrade

Copy link
Contributor

@jharris4 jharris4 left a comment

Choose a reason for hiding this comment

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

LGTM

@OskarDamkjaer OskarDamkjaer requested a review from jharris4 June 15, 2021 07:08
@OskarDamkjaer OskarDamkjaer merged commit f0546f7 into neo4j:master Jun 15, 2021
@OskarDamkjaer OskarDamkjaer changed the title Fix label issue and bump driver Bump driver Jun 22, 2021
@OskarDamkjaer OskarDamkjaer changed the title Bump driver Bump driver to 4.3.1 Jun 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants