Skip to content
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

Update docs for queue section #3223

Closed
wants to merge 1 commit into from
Closed

Conversation

hans-thomas
Copy link
Contributor

Update the docs related to using MongoDB as the database for the Laravel queue.

@hans-thomas hans-thomas requested a review from a team as a code owner November 29, 2024 11:56
@hans-thomas hans-thomas requested a review from rustagir November 29, 2024 11:56
@github-actions github-actions bot added the docs label Nov 29, 2024
Comment on lines 88 to 97

* - ``connection``
* - ``database``
- Database connection used to store jobs. It must be
a ``mongodb`` connection. The driver uses the default connection
if a connection is not specified.

* - ``collection``
* - ``table``
- Name of the MongoDB collection to store failed
jobs. The value is ``failed_jobs`` by default.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

according to the registerFailedJobServices method in the QueueServiceProvider class, the keys must be database and table instead of connections and collection.

Comment on lines -98 to -104
Then, add the service provider in your application's
``config/app.php`` file:

.. code-block:: php

MongoDB\Laravel\MongoDBQueueServiceProvider::class,

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Since it's removed in #3122

@GromNaN
Copy link
Member

GromNaN commented Nov 29, 2024

Thank you, but this doc update is already in progress in #3222. Closing as duplicate.

@GromNaN GromNaN closed this Nov 29, 2024
@hans-thomas hans-thomas deleted the issue-3000 branch November 29, 2024 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants