Botbuilder-samples: The nodejs timex resolution sample doesn't work

[dividor]

The node sample for timexresolution doesn't work. As stated in the code this is due to a dependency on a fix to the MS package.

It's been quite a while pending now and we are updating our bots for SDK 4, where datetime handling changed to timex. It would be great to be able to following a working example please, any chance we could perhaps have this issue resolved?

Thanks!

[johnataylor]

This will start working when we upgrade to the next release of the Recognizer packages.
The fix is in master in https://github.com/Microsoft/Recognizers-Text, leaving this open for now to track upgrading the dependency.

[cleemullins]

Related to: https://github.com/microsoft/botbuilder-js/issues/944

[johnataylor]

The fix to expose this method has been checked in for a while. We are waiting on the package to be published, hopefully in the next few days.

[johnataylor]

This is waiting on the JavaScript package being published. One of the examples fails because a method should be public. If we are not going to get the package out I can just pull that part of the sample. Assigning to @cleemullins to be resolved when the node package is published.

[cleemullins]

Update: We're currently having issues with the latest release of the recognizers package, and I would be surprised if this makes it into the 4.5 cycle.

[cleemullins]

Update: had a discussion with @EricDahlvang and he's looking now into the 1.1.2 to 1.1.4 updgrade of the recognizers.

[cleemullins]

The 1.1.4 update has been taken. @johnataylor can now update the sample once the 4.6 packages are pushed to production.

[johnataylor]

The samples already uses 1.1.4.

The specific dependency that matters is @microsoft/recognizers-text-data-types-timex-expression the required fix appears to have been checked in early May 2019. However, _1.1.4 was published a year ago._

[johnataylor]

Assigning to @cleemullins for tracking publishing new Node packages for Recognizers-Text.

[johnataylor]

Refer to https://www.npmjs.com/search?q=recognizers-text-data-types-timex-expression

[cleemullins]

I've followed up with the recognizers-text team about publishing a new Node set of recognizer packages. Assuming they're able to push new packages, we can then take this in 4.7.

[cleemullins]

The JS Packages should be published "Any Day" (happing now), so we should be able to resolve this.

[johnataylor]

package is still not published

[cleemullins]

Package is still not published. I followed up with the package owners today with increasingly grumpy language.

[johnataylor]

Package is still not published. Little point tracking this for R8 at this stage.