buerokratt / Training-Module

MIT License
2 stars 16 forks source link

REST service to create new Rasa responses #92

Open turnerrainer opened 1 year ago

turnerrainer commented 1 year ago

AS AN Architect I WANT to have a REST service to create new Rasa responses SO THAT there wouldn't be duplicate services for it

Acceptance Criteria

Temporary list of endpoints - https://github.com/buerokratt/Buerokratt-onboarding/issues/8 Proto to add new Rasa forms - click on "Lisa"

RaulAltmae commented 1 year ago

sample input for /rasa/responses/add

{
  "response_name": "utter_common_teenus_ilm",
  "response": {
    "utter_common_teenus_ilm": [
      {
        "condition": [
          {
            "type": "slot",
            "name": "common_teenus_ilm_asukoht",
            "value": null
          }
        ],
        "text": "Hetkel oskan ma öelda ilma Eesti raames piirkonnapõhiselt. Palun sisestage korrektne piirkonna nimi."
      },
      {
        "text": "#weather; {common_teenus_ilm_asukoht};"
      }
    ]
  }
}
RaulAltmae commented 1 year ago

Ignored

All response names in domain.yml must start with utter_ - apply automatically, if not provided in the input

turnerrainer commented 1 year ago

@RaulAltmae please link this issue with an appropriate pull request.

RaulAltmae commented 1 year ago

Pull request https://github.com/buerokratt/Training-Module/pull/168

Kristjan259 commented 7 months ago

Gives 300 error code when trying to add new response right now.

KasparMinn commented 7 months ago

PR: https://github.com/buerokratt/Training-Module/pull/316

This PR changes the functionality of the responses page slightly to fetch correct information from OpenSearch as well as correctly establish the three different searches for Rules.

Screenshots of the requests under PR link.

ValterAro commented 4 months ago

Short summary: I get error 500 when trying to add a response. How to reproduce: after logging in try to add a response at https://admin.dev.buerokratt.ee/training/training/responses

Pic:

Image

turnerrainer commented 4 months ago

According to @KasparMinn this task was previously moved to "ATR" due to deployment errors. No development done or needed.

Kristjan259 commented 4 months ago

Seems to be working as intended