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

[Messenger]  Allow InMemoryTransport to serialize message#39075

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

Conversation

@tyx
Copy link
Contributor

@tyxtyx commentedNov 13, 2020

QA
Branch?5.x
Bug fix?no
New feature?yes
Deprecations?no
TicketsFix#38893
LicenseMIT
Doc PRwip

This introduce a query parameter in dsn to enable serialization as@Nyholm has suggested in#38893

in-memory://?serialize=true

armandabric, shouze, dbrumann, and hadeli reacted with heart emojiarmandabric and Koc reacted with rocket emoji
@tyxtyx requested a review fromsroze as acode ownerNovember 13, 2020 14:16
@tyxtyxforce-pushed thefeature/in-memory-serializable-transport branch 2 times, most recently from8c31932 to916d87bCompareNovember 13, 2020 14:31
@carsonbotcarsonbot changed the title[Messenger] Allow InMemoryTransport to serialize message[Messenger]  Allow InMemoryTransport to serialize messageNov 13, 2020
@jderussejderusse added this to the5.x milestoneNov 13, 2020
@chalasr
Copy link
Member

Can you please update the component's CHANGELOG? This is for 5.3.

@tyx
Copy link
ContributorAuthor

tyx commentedNov 16, 2020

Can you please update the component's CHANGELOG? This is for 5.3.

So I need to create CHANGELOG-5.3.md ?

@tyxtyxforce-pushed thefeature/in-memory-serializable-transport branch from916d87b toe85f039CompareNovember 16, 2020 09:00
@chalasr
Copy link
Member

@tyx Nope, you need to add a new5.3.0 section at the top of thehttps://github.com/symfony/symfony/blob/5.x/src/Symfony/Component/Messenger/CHANGELOG.md

tyx reacted with thumbs up emoji

@tyxtyxforce-pushed thefeature/in-memory-serializable-transport branch frome85f039 to46a8007CompareNovember 16, 2020 13:28
@tyx
Copy link
ContributorAuthor

tyx commentedNov 16, 2020

Ok done !

@chalasr
Copy link
Member

Thank you@tyx.

tyx reacted with heart emoji

@chalasrchalasr merged commita273b71 intosymfony:5.xNov 20, 2020
@tyxtyx deleted the feature/in-memory-serializable-transport branchNovember 20, 2020 11:44
@fabpotfabpot mentioned this pull requestApr 18, 2021
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@derrabusderrabusderrabus left review comments

@jderussejderussejderusse approved these changes

@chalasrchalasrchalasr approved these changes

@srozesrozeAwaiting requested review from sroze

+1 more reviewer

@maxheliasmaxheliasmaxhelias approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Projects

None yet

Milestone

5.4

Development

Successfully merging this pull request may close these issues.

[Messenger] Add serializer usage to InMemoryTransport

7 participants

@tyx@chalasr@jderusse@derrabus@maxhelias@Nyholm@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp