-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fiqare perseo-fe improvements #419
Changes from 1 commit
c620a0a
b1eb8fa
d38ffcc
f18f17b
fb9cb75
d9d800d
e666027
5f799c9
460c75f
24bbb3f
c87e96f
26a081e
9fdc5b9
8200b45
15c1d5d
2e81912
a514588
b8b3ce2
ca2b66c
ea6a05d
aa23fed
05ac1f4
88344c8
2e1025f
e75d07d
2cd53b5
a3c1cc1
c012cdc
326a6b4
9e873b8
9b15277
0695fef
c0b8505
c11d45d
05a5208
e895b69
590de2a
53c6052
eedd341
8bda3a7
e4e9bca
01ec831
805219b
32662f4
4cade58
14a0867
ef334a6
07c164e
1c894b0
e6b731a
5d3fbc4
862b176
c68e52d
6f64b09
334a59e
6a10db1
e67403a
fe26c90
70da2e4
8ab124b
3580b61
7f7ad60
4a5b42e
fd6e450
b54bc43
c5a694f
6f05372
9a77dbb
5440640
41f9c68
75aa10d
f471ce2
14b1aa9
e090738
4db0223
ec05fb7
1bfb721
13089b0
88ad953
dd56330
e71c7e6
97bc1f3
c08843e
94fa819
57334b9
9a0e631
9e975a9
27716c8
c6604b0
f3a905a
c1cddf2
e622aa3
70a972d
bd3c7b2
aa0df1d
ac5620a
4e300a4
e8d11ac
42267ba
6e3974e
08201e7
3020aab
51271cd
ce583be
d978818
ff07d10
0865dc8
8371d53
9c80eb9
df81a41
210813b
c29e89f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -106,7 +106,7 @@ describe('Metrics', function() { | |
return callback(); | ||
}, 50); | ||
}); | ||
}, | ||
} | ||
], | ||
done | ||
); | ||
|
@@ -149,7 +149,7 @@ describe('Metrics', function() { | |
return callback(); | ||
}, 50); | ||
}); | ||
}, | ||
} | ||
], | ||
done | ||
); | ||
|
@@ -191,7 +191,7 @@ describe('Metrics', function() { | |
return callback(); | ||
}, 50); | ||
}); | ||
}, | ||
} | ||
], | ||
done | ||
); | ||
|
@@ -226,13 +226,13 @@ describe('Metrics', function() { | |
|
||
should.equal(m.services.unknownt.sum.actionEntityUpdate, 1); | ||
should.equal(m.services.unknownt.sum.okActionEntityUpdate, 0); | ||
should.equal(m.services.unknownt.sum.failedActionEntityUpdate, 1); | ||
should.equal(m.services.unknownt.sum.failedActionEntityUpdate, 0); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is going to be handled in another PR: #425 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In order not disturb @fiqare-emergya-dev (a conflict will happen when #425 gets merged), let's keep as it is now in this PR. I'll fix it in the prelanding branch (hardening/fiqare-perseo-fe-improvements-prelanding) NTC (just informative) |
||
should.equal(m.services.unknownt.sum.outgoingTransactions, 1); | ||
should.equal(m.services.unknownt.sum.outgoingTransactionsErrors, 1); | ||
should.equal(m.services.unknownt.sum.outgoingTransactionsErrors, 0); | ||
return callback(); | ||
}, 150); | ||
}); | ||
}, | ||
} | ||
], | ||
done | ||
); | ||
|
@@ -274,7 +274,7 @@ describe('Metrics', function() { | |
return callback(); | ||
}, 50); | ||
}); | ||
}, | ||
} | ||
], | ||
done | ||
); | ||
|
@@ -316,7 +316,7 @@ describe('Metrics', function() { | |
return callback(); | ||
}, 50); | ||
}); | ||
}, | ||
} | ||
], | ||
done | ||
); | ||
|
@@ -364,7 +364,7 @@ describe('Metrics', function() { | |
return callback(); | ||
}, 50); | ||
}); | ||
}, | ||
} | ||
], | ||
done | ||
); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think README.md file is not the right place to include this. In addition, it should be simplified: you have already included the needed packages in packages.json and so on, so please focus just in the usage.
In sum, It should be included in development.md, just after the last item (Prettify Code) and be like this:
<server_host>:9090/api-docs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in 210813b