Conversation
|
Size Report 1Affected Products
Test Logs |
Size Analysis Report 1This report is too large (1,243,472 characters) to be displayed here in a GitHub comment. Please use the below link to see the full report on Google Cloud Storage.Test Logs |
| /** | ||
| * Represents a vector type in Firestore documents. | ||
| * Create an instance with {@link FieldValue.vector}. | ||
| * Create an instance with `VectorValue.fromArray(array)`. |
There was a problem hiding this comment.
I think the correct link should be {@link vector}? I think when we moved from compat to modular, we took a lot of symbols that used to be on the FieldValue namespace (serverTimestamp, arrayUnion, etc) and moved them to top level exports and this fits the pattern. @MarkDuckworth - confirm?
There was a problem hiding this comment.
This is correct. Good catch @hsubox76. May be copy-pasta or just a mix up when moving between Firestore SDK.
egilmorez
left a comment
There was a problem hiding this comment.
Thanks! LG from a doc string perspective.
MarkDuckworth
left a comment
There was a problem hiding this comment.
I think we need to make the change @hsubox76 suggested. Otherwise this is good. Thank you for helping clean this up.
| /** | ||
| * Represents a vector type in Firestore documents. | ||
| * Create an instance with {@link FieldValue.vector}. | ||
| * Create an instance with `VectorValue.fromArray(array)`. |
There was a problem hiding this comment.
This is correct. Good catch @hsubox76. May be copy-pasta or just a mix up when moving between Firestore SDK.
| /** | ||
| * Represents a vector type in Firestore documents. | ||
| * Create an instance with {@link FieldValue.vector}. | ||
| * Create an instance with `{@link vector}`. |
There was a problem hiding this comment.
I don't think this is right, there shouldn't be backticks? If you use the correct format, the resulting markdown files shouldn't have the word "@link" in them anymore, it should be a markdown link (brackets then parens)
There was a problem hiding this comment.
By the way, this is the way to have something that is both a link and code formatted - you have to use the code tags here in the comment, you can't use backticks:
<code>{@link vector}</code>
There was a problem hiding this comment.
I wrapped this in <code></code> in my latest commit and it has applied both link and code formatting, but I'm not sure if the link itself is correct. When I click it, it doesn't redirect me to the vector() function in the docs. Is this expected?
There was a problem hiding this comment.
I think it should work, it comes out the same with or without the <code> tags and a lot of other existing functions seem to have the same format. In any case it's an improvement as it's not a empty space so let's try.
hsubox76
left a comment
There was a problem hiding this comment.
Sorry this review is so late.
In response to comments in cl/665934159