Skip to content

Standardization #140

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

Merged
merged 181 commits into from
Jan 31, 2025
Merged

Standardization #140

merged 181 commits into from
Jan 31, 2025

Conversation

norareidy
Copy link
Collaborator

@norareidy norareidy commented Jan 31, 2025

mongoKart and others added 30 commits December 4, 2024 16:55
(cherry picked from commit c2b53c0)

Co-authored-by: Mike Woofter <[email protected]>
* DOCSP-45170: Landing page and cleanup

* copy snooty file

* fix toc

* netlify script

* fix

* put old files in folder

* MW feedback
* DOCSP-45171: Download and Install

* snooty

* landing page

* MW feedback

* DR feedback

* fix
* DOCSP-45173: Deployment & connection string

* edits

* edit image

* SA feedback
* DOCSP-45175: Connect to MongoDB

* fixes

* edits, next steps

* JS feedback

* DR feedback
* DOCSP-45194: Retrieve data

* edits

* build errors

* api links

* JB feedback
* add files

* Update add-netlify-links.yml (#90)

* DOCSP-45076: Document that count supports different filters from countDocuments

* Test fix

* Fix

* Try again

* Never mind

* NR suggestion

* DOCSP-45203 Single Field Indexes

* Edits

* edits

* get rid of old commit work

* code spacing

* Stephanie's reiview

* Tech review code improvements

* update api links

* add a space at eof

* update listing description

* add context to ex query code comment

* style edit on prev

---------

Co-authored-by: Mike Woofter <[email protected]>
Co-authored-by: Michael Morisi <[email protected]>
mcmorisi and others added 24 commits January 17, 2025 10:47
…ections

DOCSP-45183-time-series-collections
* DOCSP-45200: Change streams

* edits

* JB feedback
DOCSP-45209: Authentication landing + Authentication mechanisms
* Configure TLS

* insecure tls

* edits

* monospace

* 1/2 MW edits

* last of MW revisions and table

* table reformat

* spacing edit

* formatting edits

* format

* edit

* MW last comments

* small change

* tech review
* DOCSP-45812 Connection Options

* edits

* more edits

* more edits

* format

* format

* RM review

* another link

* add timeout_ms

* remove deprecated options
* DOCSP-45201 Cluster monitoring

* first draft

* remove eevrything not about cluster monitoring:

* table changes

* review

* final few changes

* NR review comments

* NR comments

* look through

* tech review comments
* DOCSP-45188 Server selection

* quick add

* remove page

* spacing

* review comment

* same edit

* tech review
* DOCSP-46862 Issues & Help

* edits

* internal review

* weird build fail

* build mysteriously good again
* first draft

* revisions

* edit

* edits

* code edits

* Rea comments

* edits

* last internal comments

* tech review + add atlas search api refs

* edit to api refs

* code comment edit

* fix code example option name
* DOCSP-46650: Cleanup

* redirects

* more cleanup

* rename, test api link

* api links

* fix toc api link

* monitoring, edits

* check api links

* LM feedback

* snooty
Copy link

netlify bot commented Jan 31, 2025

Deploy Preview for docs-ruby ready!

Name Link
🔨 Latest commit e4ed3f9
🔍 Latest deploy log https://app.netlify.com/sites/docs-ruby/deploys/679d3e1b6eb69b0008883f15
😎 Deploy Preview https://deploy-preview-140--docs-ruby.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@lindseymoore lindseymoore left a comment

Choose a reason for hiding this comment

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

3 small things that could be changed, otherwise LGTM!

:dedent:
:copyable:

Additional Information
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Additional Information
API Documentation

Just to be consistent with the headings

.. contents:: On this page
:local:
:backlinks: none
:depth: 1
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
:depth: 1
:depth: 2

To make sure API documentation shows up in TOC

@@ -39,14 +41,13 @@ this, you can try the following:
host list.
- If you are connecting to Atlas, ensure that the firewall rules are configured
correctly. See the documentation on `Cluster Security <https://www.mongodb.com/docs/atlas/setup-cluster-security/>`_.
- If your application uses a forking web server, see the documentation on :ref:`Forking <forking>`.

See more on this issue here:
`RUBY-1281 <https://jira.mongodb.org/browse/RUBY-1281>`_.


``OperationFailure`` When Reading ``local`` Database With Auth On
Copy link
Contributor

Choose a reason for hiding this comment

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

An unnecessary new line after this title, but I think just because it is very long. Not sure how to fix, so I think fine.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

hm yeah I don't think I can fix that

@norareidy norareidy merged commit 88c9d5c into master Jan 31, 2025
5 checks passed
@norareidy norareidy deleted the standardization branch January 31, 2025 21:21
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.

7 participants