-
Notifications
You must be signed in to change notification settings - Fork 156
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
Fix duplication of terms #262
Conversation
…p#219 also, remove parent them, if it's removed in source
@arsendovlatyan it looks like the build failed on these changes, mind giving those a review to try and ensure this build passes? |
Resolves #325. |
@adamsilverstein @dkotter if either of you have time to review this that would be great, thanks! |
I cannot figure out why it's reporting whitespace at the end of line 447 and not reporting the warnings at line 490 that I get locally. Merging, will deal with it separately if necessary. |
Thank you very much @helen , will try to be more accurate with code and cover with tests as well. |
Fix duplication of terms with every update, in case if term slug is custom set in source website and term creation is not disabled/skipped with hooks in destination. #261