-
Notifications
You must be signed in to change notification settings - Fork 72
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
feat: Import new taxonomy dialog flow #1017
feat: Import new taxonomy dialog flow #1017
Conversation
Thanks for the pull request, @yusuf-musleh! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
f448635
to
a0ae3bc
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1017 +/- ##
==========================================
+ Coverage 92.40% 92.43% +0.02%
==========================================
Files 708 708
Lines 12523 12565 +42
Branches 2702 2760 +58
==========================================
+ Hits 11572 11614 +42
Misses 914 914
Partials 37 37 ☔ View full report in Codecov by Sentry. |
e59343a
to
ff1ac49
Compare
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.
This is a great refactor @yusuf-musleh ! It works perfectly and looks good.
There's just a couple strings missing from messages
, and one minor nit, but this will be ready for upstream review once they're addressed.
👍
- I tested this using the PR test instructions
- I read through the code
- I checked for accessibility issues by using my keyboard to navigate while testing.
-
Includes documentationN/A - User-facing strings are extracted for translation -- see inline note
cddd653
to
a57fda0
Compare
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.
Looks good! Just one small nit.
a57fda0
to
6828536
Compare
@xitij2000 It should be good to go for merge |
@yusuf-musleh 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
This PR updates the existing import tags wizard to also handle importing new taxonomies.
Description
This PR utilizes the existing import tags wizard that updates taxonomies for importing new taxonomies as well.
Supporting information
Related Tickets:
Testing instructions
Private-ref: FAL-3725