Skip to content

DOCSP-39569 changed limited to unsupported for setParameter#6275

Merged
sarah-olson-mongodb merged 1 commit into
mongodb:masterfrom
sonderdonk-mdb:DOCSP-39569
Jun 14, 2024
Merged

DOCSP-39569 changed limited to unsupported for setParameter#6275
sarah-olson-mongodb merged 1 commit into
mongodb:masterfrom
sonderdonk-mdb:DOCSP-39569

Conversation

@sonderdonk-mdb

Copy link
Copy Markdown
Contributor

DESCRIPTION

changed "this command has limited support" to "this command is unsupported" for setParameter compatibility section

STAGING

https://preview-mongodbsonderdonkmdb.gatsbyjs.io/docs/DOCSP-39569/reference/command/setParameter/#compatibility

JIRA

https://jira.mongodb.org/browse/DOCSP-39569

BUILD LOG

https://workerpool-boxgs.mongodbstitch.com/pages/job.html?collName=queue&jobId=666affc2ebce19f3148c3a21

SELF-REVIEW CHECKLIST

  • Does each file have 3-5 taxonomy facet tags?
    See the taxonomy tagging instructions and this example PR
  • Is this free of any warnings or errors in the RST?
  • Is this free of spelling errors?
  • Is this free of grammatical errors?
  • Is this free of staging / rendering issues?
  • Are all the links working?

EXTERNAL REVIEW REQUIREMENTS

What's expected of an external reviewer?

@kennethdyer kennethdyer self-requested a review June 13, 2024 16:09

@kennethdyer kennethdyer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this work @sonderdonk-mdb. Left a small nonblocking suggestion for your consideration. LGTM.

@@ -1,4 +1,4 @@
.. note::

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nonblocking Suggestion:

This is not in the scope of your project, but if you apply a two space indent to the include directive used to call this RST file, it'll render as a child of the lists item instead of breaking the list into two separate lists.

Bonus: Since the robots work from simplified HTML, I suspect that'll help their comprehension.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

spoke to Kenneth over slack, ended up not implementing

@sonderdonk-mdb

Copy link
Copy Markdown
Contributor Author

Will Pierce gave external review and approved

@sarah-olson-mongodb sarah-olson-mongodb merged commit 5e714b0 into mongodb:master Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants