Github user dszeto commented on the issue:
https://github.com/apache/incubator-predictionio/pull/292
Thank you @bansarishah ! Merging this with livedoc.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user bansarishah commented on the issue:
https://github.com/apache/incubator-predictionio/pull/292
Hi @dszeto , I have updated template to use predictionio new release
0.10.0-incubating.
I will redo this against the livedoc branch.
Please consider it.
Thank you
Github user dszeto commented on the issue:
https://github.com/apache/incubator-predictionio/pull/292
Hi @bansarishah , now that 0.10.0-incubating has released, would you mind
updating your template to use those new artifacts? Once that's done I will
merge this in.
You might
Github user bansarishah commented on the issue:
https://github.com/apache/incubator-predictionio/pull/292
I have updated template with 'PEventStore' API. Now it works on appName
instead appId and have mentioned it's use in README.md
Please consider it.
Thank you
---
If your
Github user dszeto commented on the issue:
https://github.com/apache/incubator-predictionio/pull/292
Thanks for your contribution @bansarishah ! Most of it looks great to me. A
few suggested improvement:
* highlight the need to change `appId` in `engine.json`
* or even