Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Export redirects to csv #299

Merged
merged 3 commits into from Jun 12, 2023
Merged

Export redirects to csv #299

merged 3 commits into from Jun 12, 2023

Conversation

zamanq
Copy link
Contributor

@zamanq zamanq commented Jan 14, 2023

Description of the Change

Added a new WP CLI command to export the redirects into a csv file.

How to test the Change

To test the feature, please run wp safe-redirect-manager export_csv This will create a csv file with the default name srm-redirects.csv. If you want to add a different name for the csv file then you can do something like wp safe-redirect-manager export_csv --filename=sample-redirects

*Please note that it is not necessary to provide the .csv extension for the name, it'll be appended automatically.

Changelog Entry

Added - WP-CLI command export to exporting redirects into a CSV file.

Credits

Props @zamanq @jeffpaul

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

@zamanq zamanq self-assigned this Jan 14, 2023
@jeffpaul
Copy link
Member

@zamanq are you still working on this or have you abandoned effort here such that this should be closed?

@jeffpaul jeffpaul added this to the Future Release milestone Apr 10, 2023
@zamanq
Copy link
Contributor Author

zamanq commented Apr 11, 2023

Hi @jeffpaul, Yeah I planned to work on it but then got busy with client work and eventually forgot about it, sorry. I think I'll re-visit the implementation idea in a near future. You can close this PR btw.

Thank you!

@zamanq zamanq changed the title WIP: Export redirects to csv Export redirects to csv May 24, 2023
@zamanq zamanq marked this pull request as ready for review May 24, 2023 12:01
@zamanq zamanq requested a review from a team as a code owner May 24, 2023 12:01
@zamanq zamanq requested review from peterwilsoncc and jeffpaul and removed request for a team and peterwilsoncc May 24, 2023 12:01
@jeffpaul jeffpaul requested review from a team and Sidsector9 and removed request for jeffpaul and a team May 24, 2023 12:54
Copy link
Member

@Sidsector9 Sidsector9 left a comment

Choose a reason for hiding this comment

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

Thank you for working on this, tested and works well 👍

Sidsector9

This comment was marked as duplicate.

Sidsector9

This comment was marked as duplicate.

@jeffpaul jeffpaul modified the milestones: Future Release, 2.1.0 Jun 2, 2023
@jeffpaul jeffpaul merged commit 832ca34 into develop Jun 12, 2023
11 checks passed
@jeffpaul jeffpaul deleted the feature/export-redirect-list branch June 12, 2023 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants