-
Notifications
You must be signed in to change notification settings - Fork 1.5k
DOCSP-38076 quickstart updates for 11 #2799
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
DOCSP-38076 quickstart updates for 11 #2799
Conversation
(cherry picked from commit cfb2c4c)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just requesting changes due to the question I asked
docs/quick-start/write-data.txt
Outdated
|
||
.. tip:: | ||
|
||
Skip this step if you are using Laravel 10.x since the file that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
S: I'd switch to "because" (style guide doesn't like "since")
Skip this step if you are using Laravel 10.x since the file that | |
Skip this step if you are using Laravel 10.x because the file that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! Will update.
|
||
Open the ``app.php`` file in the ``config`` directory and | ||
add the following entry into the ``providers`` array: | ||
Open the ``providers.php`` file in the ``bootstrap`` directory and add |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Q: is this the correct file name? I only see an app.php
file and a cache
folder in the bootstrap directory
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for checking.
Yes, this is the correct file and path. I think you might be looking at Laravel 10.x, but this is for 11.x. The Laravel 11.x docs refer to this file here:
https://laravel.com/docs/11.x/providers#registering-providers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
JIRA:
https://jira.mongodb.org/browse/DOCSP-38076
Staging:
Checklist