[Docs+] Getting Started With MongoDB and FastAPI#250
[Docs+] Getting Started With MongoDB and FastAPI#250rachel-mack merged 41 commits intomongodb:masterfrom
Conversation
✅ Deploy Preview for docs-pymongo ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
rustagir
left a comment
rustagir
left a comment
There was a problem hiding this comment.
I reviewed 2/3 of the PR but there are a lot of formatting and style guide issues that It would be good for you to correct before I take a second pass. Let me know if you have any questions!
There was a problem hiding this comment.
This step has a lot of indentation errors
There was a problem hiding this comment.
Since this page is now in the pymongo docs, change this link out for something that makes more sense (and use a ref)
There was a problem hiding this comment.
S: move code to file where possible
There was a problem hiding this comment.
S: fix for lots of style guide errors
rustagir
left a comment
rustagir
left a comment
There was a problem hiding this comment.
still a lot of issues to resolve, I am going OOO so perhaps another team member can pick this up!
There was a problem hiding this comment.
Since it's only two items, I'm going to leave it as a paragraph.
There was a problem hiding this comment.
S: not sure why formatting is off here - also use __ at the end of urls (2 underscores)
There was a problem hiding this comment.
Add punctuation to these list items
| - Python v3.9.0 or later | |
| - A MongoDB Atlas cluster | |
| To learn how to set up a cluster, see | |
| the :ref:`Getting Started <pymongo-get-started>`__ guide for more information. | |
| - Python v3.9.0 or later. | |
| - A MongoDB Atlas cluster. | |
| To learn how to set up a cluster, see | |
| the :ref:`Getting Started <pymongo-get-started>`__ guide for more information. |
There was a problem hiding this comment.
| Run the following command in your terminal to clone the code from the `mongodb-with-fastapi <https://github.com/mongodb-developer/mongodb-with-fastapi>`__ GitHub repository: | |
| Run the following command in your terminal to clone the code from the :github:`mongodb-with-fastapi <mongodb-developer/mongodb-with-fastapi>` GitHub repository: |
There was a problem hiding this comment.
S: needs to be indented more
There was a problem hiding this comment.
S: i think all this content isnt indented to the right level under the step
There was a problem hiding this comment.
S: this link isnt rendering properly. Consider moving this outside the code example as a note
There was a problem hiding this comment.
S: this code block indentation is off
There was a problem hiding this comment.
S: this type of "list sentence" is not style guide adherent
There was a problem hiding this comment.
S: use the api docs source constant to shorten this URL
There was a problem hiding this comment.
| The ``update_student`` route is like a combination of the | |
| ``create_student`` and the ``show_student`` routes. It receives the ``id`` | |
| of the student to update, and the new data in the JSON body. | |
| The ``update_student`` route functions similar to a combination of the | |
| ``create_student`` and the ``show_student`` routes. It receives the ``id`` | |
| of the student to update, and the new data in the JSON body. |
rustagir
left a comment
rustagir
left a comment
There was a problem hiding this comment.
Most of my comments are minor - would recommend combing through to fix small formatting issues. Also consider interlinking other pymongo guides in the relevant CRUD locations or at the end!
🔄 Deploy Preview for docs-pymongo processing
|
|
|
||
| .. code-block:: python | ||
|
|
||
| client = AsyncMongoClient(os.environ["MONGODB_URL"],server_api=pymongo.server_api.ServerApi(version="1", strict=True,deprecation_errors=True)) |
There was a problem hiding this comment.
What's the purpose behind passing server_api to the client here?
There was a problem hiding this comment.
To opt into using the stable API: https://www.mongodb.com/docs/languages/python/pymongo-driver/current/connect/connection-options/stable-api/
There was a problem hiding this comment.
Is this a standard we use for our integration tutorials? Just curious on its inclusion here.
There was a problem hiding this comment.
We recommend using Stable API if it's compatible with your server. I added a note.
* format content * formatting * toc * setup and create * reorg * steps * steps * steps * steps * resources * format tweaks * edit * title change * vale errors * snooty conflict * formatting * formatting * restructure * numbers steps * formatting * finish edits * tweak * RR feedback * original link * edits * remove wrong file * RR feedback * vale * step format * vale * testing * vale * spacing * add details * formatting * vale * RR feedback * NS feedback * python version * NS feedback * undo delete
* format content * formatting * toc * setup and create * reorg * steps * steps * steps * steps * resources * format tweaks * edit * title change * vale errors * snooty conflict * formatting * formatting * restructure * numbers steps * formatting * finish edits * tweak * RR feedback * original link * edits * remove wrong file * RR feedback * vale * step format * vale * testing * vale * spacing * add details * formatting * vale * RR feedback * NS feedback * python version * NS feedback * undo delete
4 participants
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-48895
original URL: https://www.mongodb.com/developer/languages/python/python-quickstart-fastapi/
Updating tutorial to use Pymongo Async: mongodb-developer/mongodb-with-fastapi#19
Staging Links
Self-Review Checklist