Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

T2C intent and entity value list block fix #41

Merged
merged 12 commits into from
Mar 5, 2019
Merged

T2C intent and entity value list block fix #41

merged 12 commits into from
Mar 5, 2019

Conversation

marek-danel
Copy link
Contributor

T2C was not creating nodes for conditions with following stucture (without any error):
#myIntent && @entity:()
entity_value1 response1
entity_value2 response2

The solution was embedding Adam's old but working scripts to latest version of WAW.
I managed to make it work together with current WAW. However, there might be some small changes lost by copying over old files. Can I make sure everything works fine by running some tests?

There are also some minor fixes and error output enhancements.

@marek-danel marek-danel force-pushed the marek branch 5 times, most recently from 2841141 to 1cc3b9e Compare May 31, 2018 15:38
@marek-danel marek-danel force-pushed the marek branch 4 times, most recently from 1645f97 to 59f0d2f Compare June 3, 2018 21:21
@tomasmacek
Copy link
Contributor

tomasmacek commented Jun 4, 2018

@marek-danel adding@Tereza-Pytelova a @ondrejtomas to list of reviewers as it is bigger change impacting CI and testing

@Tereza-Pytelova
Copy link
Contributor

Postponing this to the next sprint.

@Tereza-Pytelova Tereza-Pytelova modified the milestones: Sprint 21, Sprint 22 Oct 17, 2018
@Tereza-Pytelova
Copy link
Contributor

Postponing this to the next sprint.

@Tereza-Pytelova Tereza-Pytelova modified the milestones: Sprint 22, Sprint 23 Nov 1, 2018
@Tereza-Pytelova
Copy link
Contributor

@tomasmacek I'm postponing this to the Sprint 27.

@Tereza-Pytelova Tereza-Pytelova modified the milestones: Sprint 23, Sprint 27 Nov 13, 2018
@Tereza-Pytelova Tereza-Pytelova removed this from the Sprint 27 milestone Jan 23, 2019
@Tereza-Pytelova
Copy link
Contributor

Tereza-Pytelova commented Mar 5, 2019

@tomasmacek we should either delete this or merge it.

@Tereza-Pytelova Tereza-Pytelova dismissed their stale review March 5, 2019 16:14

Very, very old one.

Copy link
Contributor

@Tereza-Pytelova Tereza-Pytelova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Finally, I approve.

@Tereza-Pytelova Tereza-Pytelova merged commit bc1361a into devel Mar 5, 2019
@Tereza-Pytelova Tereza-Pytelova changed the title t2c intent and entity value list block fix T2C intent and entity value list block fix Mar 5, 2019
@Tereza-Pytelova Tereza-Pytelova deleted the marek branch March 5, 2019 16:44
@Tereza-Pytelova Tereza-Pytelova self-assigned this Mar 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Priority: high Issues with high priority (We want to add priority labels until it is implemented in github) T2C Issues connected with T2C format
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants