Add OperatorOtpChallenge resource - #447
rcabrera-py wants to merge 1 commit into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (4)
Included review availability: 4 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. WalkthroughAdds the Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Feature Suggested reviewers: Merge Risk: ⚪ Minimal · up to The new OTP challenge resource, exports, registry entry, and targeted request/response coverage have no identified merge-blocking risk. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #447 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 58 59 +1
Lines 1297 1311 +14
=========================================
+ Hits 1297 1311 +14
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
|
Closing: no Python consumer needs the authorization OTP challenge today. The company portal uses cuenca-js, so this would be speculative API surface. Reopen when a Python service actually authorizes payment orders. |
Summary
OperatorOtpChallengemodel and public exportsPOST /operator-otp-challengesid,expires_at, andemail_hintVerification
Made with Cursor
Summary by CodeRabbit