Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Messenger] Add support for the DelayStamp in InMemoryTransport #49725

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 19, 2023
Merged

[Messenger] Add support for the DelayStamp in InMemoryTransport #49725

merged 1 commit into from
Mar 19, 2023

Conversation

fabpot
Copy link
Member

@fabpot fabpot commented Mar 18, 2023

Q A
Branch? 6.3
Bug fix? no
New feature? yes
Deprecations? no
Tickets n/a
License MIT
Doc PR n/a

@carsonbot carsonbot added this to the 6.3 milestone Mar 18, 2023
@fabpot fabpot force-pushed the in-memory-messenger-delay-stamp-support branch from 29511ca to 3bd8602 Compare March 18, 2023 16:18
OskarStark
OskarStark reviewed Mar 19, 2023
View reviewed changes
src/Symfony/Component/Messenger/Transport/InMemory/InMemoryTransport.php
$this->serializer = $serializer;
public function __construct(
private ?SerializerInterface $serializer = null,
private ?ClockInterface $clock = null,
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need this as require dev dep? Or is it already available?

@fabpot fabpot force-pushed the in-memory-messenger-delay-stamp-support branch from 3bd8602 to c57c971 Compare March 19, 2023 13:28
@fabpot fabpot force-pushed the in-memory-messenger-delay-stamp-support branch from c57c971 to 1025f35 Compare March 19, 2023 13:32
@fabpot fabpot merged commit 3276a3d into symfony:6.3 Mar 19, 2023
@fabpot fabpot deleted the in-memory-messenger-delay-stamp-support branch March 19, 2023 13:35
@fabpot fabpot mentioned this pull request Mar 19, 2023
Closed
@fabpot fabpot mentioned this pull request May 1, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@OskarStark OskarStark OskarStark left review comments

@nicolas-grekas nicolas-grekas nicolas-grekas approved these changes

@welcoMattic welcoMattic welcoMattic approved these changes

Assignees
No one assigned
Labels
Feature Messenger Status: Reviewed
Projects
None yet
Milestone
6.3
Development

Successfully merging this pull request may close these issues.
5 participants
@fabpot @nicolas-grekas @welcoMattic @OskarStark @carsonbot