update risk reasons in factory body example#1267
Conversation
Deploying dev-site with
|
| Latest commit: |
d7516b4
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://f7957584.dev-site-4ua.pages.dev |
| Branch Preview URL: | https://andram-update-factors-body-e.dev-site-4ua.pages.dev |
| "reason": "The number of distinct Issuer ID Numbers found in the | ||
| velocity check on IP address raised the overall risk score" |
There was a problem hiding this comment.
This is invalid JSON, that's why it is highlighted in red. There can't be a line break in the middle of a string. Is it possible to put it all on one line?
| "reason": "The number of distinct Issuer ID Numbers found in the | |
| velocity check on IP address raised the overall risk score" | |
| "reason": "The number of distinct Issuer ID Numbers found in the velocity check on IP address raised the overall risk score" |
There was a problem hiding this comment.
I think I understand that this is for formatting/display purposes. The problem is that if a developer copies this into a tool or something expecting valid JSON, it won't work.
If y'all want it displayed in a particular format, scotty can probably achieve that with CSS. I can create a linear issue for that purpose.
| "reason": "The local time of day of the request raised the overall | ||
| risk score" |
There was a problem hiding this comment.
| "reason": "The local time of day of the request raised the overall | |
| risk score" | |
| "reason": "The local time of day of the request raised the overall risk score" |
| "reason": "The email domain being recently seen for the first time | ||
| in the minFraud network raised the overall risk score" |
There was a problem hiding this comment.
| "reason": "The email domain being recently seen for the first time | |
| in the minFraud network raised the overall risk score" | |
| "reason": "The email domain being recently seen for the first time in the minFraud network raised the overall risk score" |
| "reason": "minFraud network activity of the phone number lowered | ||
| the overall risk score" |
There was a problem hiding this comment.
| "reason": "minFraud network activity of the phone number lowered | |
| the overall risk score" | |
| "reason": "minFraud network activity of the phone number lowered the overall risk score" |
There was a problem hiding this comment.
Thanks @mm-jpoole i will put it on one line and resubmit. the line breaks are totally on me!
There was a problem hiding this comment.
Was that done to make it display in a certain way on the page? If so, feel free to create a linear issue and we can likely adjust the formatting via CSS.
There was a problem hiding this comment.
yes, it was and it was all me! i don't think this error is anywhere else so we're likely okay to leave as-is :) unless you want to get rid of the sideways scroll that exists now. I have no objection either way.
2 participants
No description provided.