-
Notifications
You must be signed in to change notification settings - Fork 53
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
Classify content within existing Taxonomy terms #620
Conversation
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.
Suggestion
The popup would show the notice,
classifai/src/js/gutenberg-plugin.js
Line 266 in e9198e5
'Note that the lists above include any pre-existing terms from this %s.', |
We should also tweak this notice when the existing terms-based classification setting is enabled. We can tweak to something like:
'Note that the lists above include only pre-existing terms from this %s. You can change this classification method in settings.'
and add a link on the 'settings' text to the settings page.
I think |
@iamdharmesh @dkotter one item I noticed during UAT today was perhaps a copy tweak, sharing here to see if this resonates with either of you for a quick commit to change before merging this in...
|
Description of the Change
The PR introduces a new setting option in content classification (IBM Watson NLU) to classify content within existing taxonomy terms. By default, it will adhere to the current behavior, recommending and adding new terms to the post. When the user chooses "Only classify based on existing terms" in the classification method, it will recommend terms only from the existing terms.
Closes #592
How to test the Change
Tools
>ClassifAI
>Language Processing
>IBM Watson
Changelog Entry
Credits
Props @jeffpaul @iamdharmesh
Checklist: