Problems after upgrade to v. 2.2

General
1

Dear all,
Till today I was having v.1.3.x and it was working fine. SInce I wanted to have Monaco editor, I upgraded to v.2.2.
After upgrade everything seems fine. ALso all the flows were working until, I entered into some functions. I didn't change anything, but after deploy, nodes are not deployed any more. I am getting errors, they are not configured correctly.
Any idea?
This is very frustrating for me. Can I downgrade it?

2

Do you get any red triangles over those nodes .. meaning that they are not configured correctly ?
What are those nodes and can you expand a little of what errors you get ?
Maybe a screenshot of the errors can be helpful and if its a Function node that is giving the error .. share the code to check if the syntax is correct. (it could be possible that Monaco is less forgiving on some Javascript syntax errors than Ace - the previous editor)

3

Aha,
Yes, I am getting a triangle. I am getting them on Function node.
The problem is, that I don't know where the error is.
But I went step by step, And I realized that this piece of code is problematic:

d = new Date,
dformat = [d.getMonth()+1,
    d.getDate(),
    d.getFullYear()].join('/')+' '+
    [d.getHours(),
    d.getMinutes(),
    d.getSeconds()].join(':');
    dtstmp = new Date().toString();

The error is in first line.

4

you have to define a new variable with let or var

let d = new Date,
    dformat = [d.getMonth() + 1,
    d.getDate(),
    d.getFullYear()].join('/') + ' ' +
        [d.getHours(),
        d.getMinutes(),
        d.getSeconds()].join(':');

let dtstmp = new Date().toString();

Exactly .. Monaco underlines the error :wink:

image

1 Like
5

Aha...you're right.
I didn't define d as variable. After that, it is working. I had this piece of code in majority of problematic functions. It looks like "ace" ignored that kind of mistakes.
Thank you very much!

2 Likes
6

Yes, Monaco is very much more aggressive (correctly so) at spotting coding errors. Truth is that ACE is rather long in the tooth now.

7

This topic was automatically closed 14 days after the last reply. New replies are no longer allowed.