8000 [Asset] Remove deprecated RemoteJsonManifestVersionStrategy by mbabker · Pull Request #41300 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[Asset] Remove deprecated RemoteJsonManifestVersionStrategy #41300

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
May 23, 2021
Merged

[Asset] Remove deprecated RemoteJsonManifestVersionStrategy #41300

merged 1 commit into from
May 23, 2021

Conversation

@mbabker
Copy link
Contributor
@mbabker mbabker commented May 19, 2021
Q A
Branch? 6.0
Bug fix? no
New feature? no
Deprecations? no
Tickets -
License MIT
Doc PR -

Removes the deprecated RemoteJsonManifestVersionStrategy class from the Asset component and corresponding service definition from the FrameworkBundle

@carsonbot carsonbot added this to the 6.0 milestone May 19, 2021
@mbabker mbabker force-pushed the asset-deprecations branch from ce9b188 to 2d886f5 Compare May 19, 2021 23:06
jderusse
jderusse reviewed May 19, 2021
View reviewed changes
@mbabker mbabker force-pushed the asset-deprecations branch from 2d886f5 to cf88b06 Compare May 19, 2021 23:41
@nicolas-grekas
Copy link
Member
nicolas-grekas commented May 23, 2021

Thank you @mbabker.

@nicolas-grekas nicolas-grekas merged commit 0a01d29 into symfony:6.0 May 23, 2021
@mbabker mbabker deleted the asset-deprecations branch May 23, 2021 13:32
@fabpot fabpot mentioned this pull request Nov 5, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jderusse jderusse jderusse left review comments

Assignees

No one assigned

Labels

Asset Status: Needs Review

Projects

None yet

Milestone

6.0

Development

Successfully merging this pull request may close these issues.

4 participants

@mbabker @nicolas-grekas @jderusse @carsonbot
0