Skip to content

Add --file_name=<name> argument for wp media import#187

Merged
danielbachhuber merged 4 commits intowp-cli:mainfrom
sandeshjangam:media-slug-option
Nov 10, 2023
Merged

Add --file_name=<name> argument for wp media import#187
danielbachhuber merged 4 commits intowp-cli:mainfrom
sandeshjangam:media-slug-option

Conversation

@sandeshjangam
Copy link
Contributor

Added a slug option for media

Issue - #186

@sandeshjangam sandeshjangam requested a review from a team as a code owner November 8, 2023 06:41
Copy link
Member

@danielbachhuber danielbachhuber left a comment

Choose a reason for hiding this comment

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

Thanks for the pull request, @sandeshjangam !

Can you include some functional tests? Here's the documentation on how to do so: https://make.wordpress.org/cli/handbook/contributions/pull-requests/#functional-tests

@danielbachhuber
Copy link
Member

Can you include some functional tests? Here's the documentation on how to do so:

@sandeshjangam If it's helpful, here's a video I put together too: wp-cli/wp-cli#5858

@sandeshjangam
Copy link
Contributor Author

@danielbachhuber Done with the functional test. Thanks for the reference.

@danielbachhuber danielbachhuber changed the title Added slug option for media Add --file_name=<name> argument for wp media import Nov 10, 2023
@danielbachhuber danielbachhuber added the command:media-import Related to 'media import' command label Nov 10, 2023
@danielbachhuber danielbachhuber added this to the 2.0.21 milestone Nov 10, 2023
Co-authored-by: Alain Schlesser <alain.schlesser@gmail.com>
@danielbachhuber danielbachhuber dismissed schlessera’s stale review November 10, 2023 21:54

Changes were addressed

Copy link
Member

@danielbachhuber danielbachhuber left a comment

Choose a reason for hiding this comment

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

Thanks for your work on this, @sandeshjangam !

@danielbachhuber danielbachhuber merged commit 4950ed4 into wp-cli:main Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

command:media-import Related to 'media import' command

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants