Skip to content
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

chore(dev): merge qna in codebase #5170

Merged
merged 26 commits into from
Aug 17, 2021
Merged

chore(dev): merge qna in codebase #5170

merged 26 commits into from
Aug 17, 2021

Conversation

allardy
Copy link
Member

@allardy allardy commented Jul 7, 2021

This PR is linked to botpress/studio#29

One step closer to the goal of having all bot management done on the studio.

  1. Move all routes related to NLU management (create intent/entities, update, delete) to the studio ("get" routes are duplicated on the studio and the core for the moment)
  2. Move the QNA UI over to the studio codebase (the frontend and associated backend)
  3. Create a QNA Service on the core to handle get queries (and ndu intent actions) and process the middleware
  4. Move studio-related logic over there (onFlowChanged, onFlowRenamed, onTopicChanged...)

Since the NLU module is still pretty integrated, i've added a small workaround so the studio can call the core and ask to check if training is required. Eventually, the goal is to move all the train logic over to the studio.

@allardy allardy changed the base branch from ya-reorg to master August 5, 2021 20:05
@allardy allardy marked this pull request as ready for review August 12, 2021 14:03
@allardy allardy requested a review from a team August 12, 2021 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants