superscriptjs / superscript

A dialogue engine for creating chat bots
http://superscriptjs.com
MIT License
1.65k stars 209 forks source link

Make extraScope can access in trigger rules #382

Open golfapipol opened 6 years ago

golfapipol commented 6 years ago

uncommnet line:156 in helpers.js that make extraScope cannot use in trigger rules use options.system.extraScope instead of old one

Description

uncommnet line:156 in helpers.js that make extraScope cannot use in trigger rules use options.system.extraScope instead of old one

Motivation and Context

I want to use extraScope in trigger rules and found this commented code here

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

Checklist:

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.07%) to 80.565% when pulling 04d197858b00ad6d14b7067fb3eda4bff62fba98 on golfapipol:make-extraScope-work-in-getReply into d7cf87a4b3dc044526861844c3684ddf50e24184 on superscriptjs:master.

golfapipol commented 6 years ago

I think we should change filterScope.message_props to filterScope.extraScope or not?