Update H1/H2 headings throughout #78
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Update the About link to correct the 404 error. Then add a Social Media link to Vectara's Discord channel.
Add doc_meta example and update curl command syntax with single quotes around URL
Update doc_metadata example
Fix example for doc_meta
Updated a few statements based on team discussion around indexing
80 char spacing
Users need to be careful so that they do not accidentally share their keys or tokens in public channels. I added this note to three topics that discuss API keys and OAuth tokens.
Updated spacing to resolve conflict
Update information about the Vectara platform along with information about common use cases Update sidebar to reflect changes
Updated H1 headings to be H2 or H3 where appropriate and also fixed some typos
✅ Deploy Preview for luxury-shortbread-acee05 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Signed-off-by: Paul Wozniczka <[email protected]>
cjcenizal
approved these changes
Aug 23, 2023
There was a problem hiding this comment.
Looks great!! Had a couple non-blocking comments.
Also I noticed that the first page's "Not sure where to start?" heading appears as a child of "Language Agnostic", which seems like a mistake because the two aren't related. Should "Not sure where to start?" be an h2?
www/docs/common-use-cases/filtering-by-metadata/built_in_filterable_metadata.md
Outdated
Show resolved
Hide resolved
| @@ -24,7 +24,7 @@ Valid filter expressions for this would be something like: | |||
| * `part.lang = 'deu'` | |||
| * `part.lang = 'eng' OR part.lang = 'deu'` | |||
|
|
|||
| ## part.is_title | |||
| ## part.is_title field | |||
There was a problem hiding this comment.
Thank you! I made the suggested updates in my latest commit
Updated headings based on feedback
|
Moved from draft to ready for review |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixed headings throughout the docs so that pages would not have multiple H1's