-
Notifications
You must be signed in to change notification settings - Fork 875
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
treat the ns=
query param as the namespace name if it is present
#2060
Conversation
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.
Awesome!
Since this is a breaking change, we should add a change log entry that calls this out. Do you mind doing this?
I also want your verbal (or rather written) nod that you tested this manually. Thanks.
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.
Basically LGTM with a couple suggestions. I also would like Sebastian to look at this again since he did the initial review and he owns the RTDB SDK at this point. :-)
webSocketOnly | ||
webSocketOnly, | ||
'', | ||
parsedUrl.namespace != parsedUrl.subdomain |
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.
When using literals or boolean expressions as function arguments, it's often helpful to comment them for the reader, something like:
webSocketOnly,
/*persistenceKey=*/ '',
/*withQueryParam=*/ parsedUrl.namespace != parsedUrl.subdomain
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.
LGTM
I tested this manually. |
…2060) * treat `ns=` query param as the namespace name if it is present * [AUTOMATED]: Prettier Code Styling * record ns queryparam override in RepoInfo class * [AUTOMATED]: Prettier Code Styling * update database test suite * more descriptive RepoInfo member name * Add to changelog
No description provided.