DOCSP-17260 Direct Text Search traffic to Atlas Search #5946
Conversation
kanchana-mongodb
force-pushed
the
DOCSP-17260
branch
6 times, most recently
from
f7b279a to
0b04fa2
Compare
kanchana-mongodb
force-pushed
the
DOCSP-17260
branch
from
0b04fa2 to
dfa34c5
Compare
There was a problem hiding this comment.
Hey @kanchana-mongodb , left some suggestions. Mostly minor, but did have one refactoring idea to consider.
Jeff
source/text-search.txt
Outdated
|
|
||
| Language Support | ||
| ---------------- | ||
| Users running MongoDB software locally can find information on the |
There was a problem hiding this comment.
[nit] Suggest removing "software".
source/text-search.txt
Outdated
| Language Support | ||
| ---------------- | ||
| Users running MongoDB software locally can find information on the | ||
| legacy text search here: |
There was a problem hiding this comment.
[suggestion]
Can we reword this to be a bit more active, as in:
"To learn more about legacy text search for self-managed deployments, see: "
source/core/link-text-indexes.txt
Outdated
|
|
||
| .. code-block:: javascript | ||
|
|
||
| db.stores.insert( |
There was a problem hiding this comment.
the "insert" method is deprecated. Please make this "insertMany".
|
|
||
| For example you can run the following in :binary:`~bin.mongosh` to | ||
| allow text search over the ``name`` and ``description`` fields: | ||
| You can run the following in :binary:`~bin.mongosh` to allow text |
There was a problem hiding this comment.
Suggest removing "You can" and being more direct. "Run the following command in...
| @@ -39,6 +39,51 @@ operators, including restrictions and behavior, see: | |||
|
|
|||
| - :expression:`$meta` projection operator | |||
|
|
|||
| Exact Phrase | |||
There was a problem hiding this comment.
I'm a little concerned about the discoverability of these examples. It seems slightly out of place to have them tucked under the "Query Framework" section. These seem like critical use cases we should be highlighting to users.
What if we:
- Moved these examples to the Text Indexes page
- Renamed the Text Indexes page to something like "[Perform|Configure] Text Search", to help avoid confusion with the other Text Indexes page.
- Phrase these section titles more like tasks, so it's clearer what users can do. For example, make this first section "Search for an Exact Phrase"
There was a problem hiding this comment.
Thanks @kanchana-mongodb ! I really like the new structure. Just some quick thoughts on the headings / organization and then I think this is all set.
source/core/link-text-indexes.txt
Outdated
|
|
||
| .. default-domain:: mongodb | ||
|
|
||
| .. include:: /includes/fact-text-index.rst | ||
| Overview |
There was a problem hiding this comment.
Suggest removing the "Overview" heading. I don't think it adds much here.
source/core/link-text-indexes.txt
Outdated
|
|
||
| .. _text-index-eg: | ||
|
|
||
| Example |
There was a problem hiding this comment.
Maybe make this "Examples" since there are several examples in this section.
|
Text Search page LGTM However on the "Text Indexes" / "Perform a Text Search" and "Text Search Operators" pages can we also update these with similar framing, namely that "MongoDB offers a premium full-text search solution, MongoDB Atlas Search, for MongoDB Atlas users and a legacy text search capability for self-managed deployments up top." and incorporate the word "Legacy" in the titles "Text Search Operators (Legacy)" and "Perform a Text Search (Legacy)" or similar? |
@andrewad Updated. Can you please take a look and let me know if changes look ok? Thanks! |
|
LGTM but one final comment for all three pages: |
|
Kanchana said Jeff was okay with all the changes, so will squash and merge. |
* DOCSP-17260 Direct Text Search traffic to Atlas Search (#5946) * DOCSP-17260 Direct Text Search traffic to Atlas Search * DOCSP-17260 updates for copy review feedback * DOCSP-17260 2nd round of updates for copy review feedback * DOCSP-17260 updates for review feedback * DOCSP-17260 added link to search and intro to two more pages * DOCSP-17260 addressing Jenny's feedback comments * fix text search url Co-authored-by: kanchana-mongodb <54281287+kanchana-mongodb@users.noreply.github.com>
* DOCSP-27736-sql-error * DOCSP-27736-sql-error * DOCSP-27736-sql-error * DOCSP-27736-sql-error * DOCSP-27736-sql-error * DOCSP-27736-sql-error * DOCSP-27736-sql-error * DOCSP-27736-sql-error * DOCSP-27736-sql-error * DOCSP-27736-sql-error --------- Co-authored-by: jason-price-mongodb <jshfjghsdfgjsdjh@aolsdjfhkjsdhfkjsdf.com>
Uh oh!
There was an error while loading. Please reload this page.