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
Jscbc 831 named scopes first #119
Jscbc 831 named scopes first #119
Conversation
Low hanging fruit: - version-server attribute - beta refs
(fuller update in other ticket)
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, just minor comments added!
@@ -39,7 +39,7 @@ Note: We have used the `-y` flag to take the initialization defaults. To change | |||
|
|||
[source,json,indent=0] | |||
---- | |||
include::../examples/getting-started/package.json[tag=start-using-package-json] | |||
include::../examples/getting-started/package.json[tags=**] |
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.
Just a question for my benefit, what does [tags=**]
do?
I vaguely remember you mentioned Antora tags in the package.json
, were they causing issues or just seemed odd?
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.
it shows everything, even if not tagged (as .json doesn't have comments so can't be tagged and remain valid)
This only worked for the KV APIs though -- Query, Search, Views, and other services exposed their JSON rows/hits in different ways. | ||
All of this has been unified in SDK 3 under a single concept: serializers and transcoders. | ||
|
||
By default, all KV APIs transcode to and from JSON -- you can also provide java POJOs which you couldn't in the past. | ||
By default, all KV APIs transcode to and from JSON -- you can also provide java POJOs which you couldn't in the past. |
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.
nodejs
instead of java
.
No description provided.