tock icon indicating copy to clipboard operation
tock copied to clipboard

[FAQ] Enhancement and updates

Open djowalker opened this issue 2 years ago • 8 comments

  • Full frontend refactoring about FAQ
  • Introduce FAQSettings to use redirection (for example satisfaction on all FAQ)
  • More unit tests
  • Fixing search trouble when a classified sentences is deleted, refactoring and testing the code for aggragation search and add a filter ClassifiedSentences status because the BuildWorker update the sentences each 1s.
  • Adding a fix to work on DocumentDB when using aggregation pipeline request for FAQ
  • Show MISSING_UTTERANCE in frontend when a FAQ has no more classified sentences associated.
  • FAQ will use story name and description instead of the actual intention name and description : to avoid trouble with shared intents
  • Some more unit test in frontend
  • Add Display Unknown sentences only in FAQ Training

djowalker avatar Jun 14 '22 09:06 djowalker

@djowalker is it still a draft?

vsct-jburet avatar Jun 22 '22 14:06 vsct-jburet

@vsct-jburet Yes, there are others new things to merge in it.

djowalker avatar Jun 22 '22 14:06 djowalker

@vsct-jburet : There are also database update on faq_definition We add an applicationId in the collection.

djowalker avatar Jun 23 '22 07:06 djowalker

@vsct-jburet : the PR should be ok for this week.

djowalker avatar Jun 27 '22 07:06 djowalker

3 more things that are going to be added in this PR :

  • FAQ will use story label instead of the actual intention name
  • Some more unit test in frontend
  • Add Display Unknown sentences only in FAQ Training

djowalker avatar Sep 01 '22 12:09 djowalker

  • FAQ will use story label instead of the actual intention name : will be added this week.

djowalker avatar Sep 06 '22 12:09 djowalker

@vsct-jburet ready for review

djowalker avatar Sep 08 '22 16:09 djowalker

@vsct-jburet : There was a trouble with the tests in Messenger, but you upgrade the version recently so i added the fix from Master in this branch

djowalker avatar Sep 16 '22 14:09 djowalker

LGTM, thank you !

vsct-jburet avatar Sep 26 '22 08:09 vsct-jburet