-
Notifications
You must be signed in to change notification settings - Fork 92
Migrate term from a taxonomy to another one #234
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
Conversation
|
@Mte90 Are you able to write some tests for this? |
|
Yes I can do it but I am not sure about the warning (specifically about the text messages) printed in the console. |
|
Let me do a quick review of the text messages then so you now what final strings to test against... |
Co-Authored-By: Mte90 <mte90net@gmail.com>
|
I will work on the other changes and tests in the next days. |
schlessera
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few small string changes, and then we're good to merge.
apply second review Co-Authored-By: Mte90 <mte90net@gmail.com>
|
Thanks for the PR, @Mte90! |
The new command is avalaible!
I need only to do the unit tests but I will work on them but before to do that I want a feedback about the warning in the code.
Example output: