Add agent model name to request query

[Oceania2018]

localhost:5000/train?project=my_project&model=my_model_name

[paschmann]

@Oceania2018 - please validate why you need the agent name included in the request? Per the Rasa NLU HTTP API Docs, there is no param needed? https://www.rasa.com/docs/nlu/http/

[Oceania2018]

@paschmann Please see the note from HTTP API Docs, "The server will automatically generate a name for the trained model. If you want to set the name yourself, call the endpoint using localhost:5000/train?project=my_project&model=my_model_name". Actually, It's better if we change it to "&model=" + req.query.name, I just follow Rasa UI pattern, even it might not make sense.

I'm trying to make Rasa UI work with another bot engine BotSharp. It still work if you don't merge this change. But I noticed that Rasa UI already generated a model name from frontend. So I consider that we might use it.

[paschmann]

Thanks for the response @Oceania2018 - but please notice in your pull request your query param is called agent, it should be model, please update and I will merge. Thanks again for your commit/help.

global.rasanluendpoint + "/train?project=" + req.query.project + "&agent=" + req.query.name

[Oceania2018]

Do you want me change the 40th line logRequest(req, "train", {project: req.query.project, agent: req.query.name, data: req.body}); to logRequest(req, "train", {project: req.query.project, model: req.query.name, data: req.body});

[paschmann]

@Oceania2018 - no, per the Rasa NLU HTTP API docs, this line should be model: https://github.com/paschmann/rasa-ui/pull/105/files#diff-9a38e7260de0e7afb6752be4607ed39cR53

[Oceania2018]

Current it's agent logRequest(req, "train", {project: req.query.project, agent: req.query.name, data: req.body}), not model.

[Oceania2018]

@paschmann Hi Paul, are you going to merge the pull request?