Skip to content

Risk score reasons#152

Merged
ugexe merged 2 commits intomainfrom
risk-reasons-beta
Aug 30, 2024
Merged

Risk score reasons#152
ugexe merged 2 commits intomainfrom
risk-reasons-beta

Conversation

@marselester
Copy link
Copy Markdown
Contributor

No description provided.

@marselester marselester force-pushed the risk-reasons-beta branch 2 times, most recently from de88d40 to 5b5b28a Compare August 26, 2024 18:09
Copy link
Copy Markdown
Contributor

@ugexe ugexe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't finished reviewing most of this so I may have more comments in the future. But I did notice the changelog needs some tweaks.

Comment thread HISTORY.rst Outdated
Copy link
Copy Markdown
Contributor

@ugexe ugexe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor comments, but otherwise looks good!

Comment thread minfraud/models.py Outdated
Comment thread minfraud/models.py
Comment thread tests/data/factors-response.json Outdated
Comment thread tests/test_models.py
Comment thread minfraud/models.py Outdated
Comment thread HISTORY.rst Outdated
@marselester marselester force-pushed the risk-reasons-beta branch 2 times, most recently from 8442cd5 to 3e23e94 Compare August 30, 2024 18:40
@ugexe ugexe merged commit 18162a4 into main Aug 30, 2024
@ugexe ugexe deleted the risk-reasons-beta branch August 30, 2024 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants