Wish List for forecast io node

39 views
Skip to first unread message

Jon Murphy

unread,
Aug 29, 2016, 11:29:11 PM8/29/16
to Node-RED
Since we’re talking about forecast.io…   
sorry I couldn’t resist    ;-)

Please add this to the future enhancements list for the forecast io node: 
  •  checkboxes for each exclude option.
    • This reduces a “currently” request (msg.data.currently) from 25,000+ characters to 550 characters.
    • error with no api forecast send if all check boxes are unchecked

Dave C-J

unread,
Aug 30, 2016, 1:58:25 AM8/30/16
to node...@googlegroups.com

We do accept pull requests 😀
(As long as a class is in place)

Dave C-J

unread,
Aug 30, 2016, 1:59:05 AM8/30/16
to node...@googlegroups.com

CLA in place not class...

Jon Murphy

unread,
Aug 30, 2016, 4:20:32 PM8/30/16
to Node-RED
Excellent point!  I'm learning javascript and I am positive you would want my code!  And I'm fairly dependent on the kindness of others.   I did do my second git pull request last month but it was for a bad link.  I'll get there!

On Tuesday, August 30, 2016 at 12:59:05 AM UTC-5, Dave C-J wrote:

CLA in place not class...

Reply all
Reply to author
Forward
0 new messages