-
Notifications
You must be signed in to change notification settings - Fork 115
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 onImport "from" option #54
Conversation
This is supposed to fix it already 5e2169d |
Hum maybe I should made the same in the plugin itself. |
Any tests to push to confirm the fix ? |
@MoOx I can verify that 5e2169d does not fix the issue. I currently have version 6.0.0 installed and I'm still getting the following output:
The fix in this PR gives me the desired/expected output:
I'll have to investigate further how to write a test for this. |
You current PR will break from option for people that specify it by hand, let my try something else. |
OK! |
Can you try |
That works! Thanks! |
I guess you can close this PR along with #44 when you merge |
Fixes #44