close
Skip to content

Remove sfwobject and swfupload#3043

Closed
remyperona wants to merge 2 commits intoWordPress:trunkfrom
wp-media:52699
Closed

Remove sfwobject and swfupload#3043
remyperona wants to merge 2 commits intoWordPress:trunkfrom
wp-media:52699

Conversation

@remyperona
Copy link

Remove swfobject and swfupload from codebase since Flash is unsupported now

Trac ticket: https://core.trac.wordpress.org/ticket/52699


This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.

@remyperona remyperona changed the base branch from master to trunk July 29, 2022 19:20
Copy link
Contributor

@audrasjb audrasjb left a comment

Choose a reason for hiding this comment

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

The PR is ok on my side. Adding needs-dev-note and add-to-field-guide keyword.

@audrasjb
Copy link
Contributor

AH. Hmm no we need a PR udpate as this one has conflicts with current trunk.

@github-actions
Copy link

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props tabrisrp, audrasjb.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@github-actions
Copy link

github-actions bot commented Jun 5, 2025

A commit was made that fixes the Trac ticket referenced in the description of this pull request.

SVN changeset: 60281
GitHub commit: 32469f1

This PR will be closed, but please confirm the accuracy of this and reopen if there is more work to be done.

@github-actions github-actions bot closed this Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants