Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Add documentation about RedisTagAwareAdapter#14081

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
OskarStark merged 1 commit intosymfony:4.4froml-vo:add_redis_tag_aware_4.4
Aug 26, 2020

Conversation

@l-vo
Copy link
Contributor

No description provided.

@l-vo
Copy link
ContributorAuthor

@OskarStark threads resolved, thank you 🙂

@l-vol-vo requested a review fromOskarStarkAugust 26, 2020 07:14
When using the `Predis`_ library some additional Predis-specific options are available.
Reference the `Predis Connection Parameters`_ documentation for more information.

.. _redis-tag-aware:
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
.. _redis-tag-aware:
.. _redis-tag-aware-adapter:


..note::

When using a Redis backend, consider using:ref:`RedisTagAwareAdapter<redis-tag-aware>` which is optimized for this purpose.
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
When using a Redis backend, consider using:ref:`RedisTagAwareAdapter<redis-tag-aware>` which is optimized for this purpose.
When using a Redis backend, consider using:ref:`RedisTagAwareAdapter<redis-tag-aware-adapter>`
which is optimized for this purpose.

@OskarStarkOskarStark added this to the4.4 milestoneAug 26, 2020
@OskarStark
Copy link
Contributor

OskarStark commentedAug 26, 2020•
edited
Loading

@l-vo after my last two comments this is ready to merge 🙏 thank you for your patience

@OskarStark
Copy link
Contributor

I added my last two comments while merging 👍 Thank you!

@OskarStarkOskarStark merged commite5cbc77 intosymfony:4.4Aug 26, 2020
OskarStark added a commit that referenced this pull requestAug 26, 2020
OskarStark added a commit that referenced this pull requestAug 26, 2020
* 4.4:  Minor: Rename ref. refs#14081  Add documentation about RedisTagAwareAdapter
OskarStark added a commit that referenced this pull requestAug 26, 2020
* 5.1:  Minor: Rename ref. refs#14081  Add documentation about RedisTagAwareAdapter
OskarStark added a commit that referenced this pull requestAug 26, 2020
This PR was merged into the master branch.Discussion----------RedisTagAwareAdapter integrationDocumentation aboutsymfony/symfony#36596This PR contains documentation on `RedisTagAwareAdapter` (introduced in 4.3) **and** its integration in the fullstack framework (introduced in 5.2).I created another PR (#14081) with only the documentation about `RedisTagAwareAdapter` that targets 4.4.Close#14065Commits-------883dc98 RedisTagAwareAdapter integration
@l-vol-vo deleted the add_redis_tag_aware_4.4 branchAugust 26, 2020 10:45
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@OskarStarkOskarStarkOskarStark requested changes

+1 more reviewer

@maxheliasmaxheliasmaxhelias approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Projects

None yet

Milestone

4.4

Development

Successfully merging this pull request may close these issues.

4 participants

@l-vo@OskarStark@maxhelias@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp