- 
                Notifications
    
You must be signed in to change notification settings  - Fork 10
 
feat(*): add plaid bank bridge connector and boilerplate #499
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
Conversation
| 
          
 Important Review skippedMore than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review. 101 files out of 230 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits. You can disable this status message by setting the  ✨ Finishing Touches
 🧪 Generate unit tests
 Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit: 
 SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
 Other keywords and placeholders
 CodeRabbit Configuration File (
 | 
    
        
          
                internal/api/services/payment_service_users_complete_link_flow.go
              
                Outdated
          
            Show resolved
            Hide resolved
        
      
          Codecov Report❌ Patch coverage is  Additional details and impacted files@@            Coverage Diff             @@
##             main     #499      +/-   ##
==========================================
- Coverage   69.56%   68.74%   -0.82%     
==========================================
  Files         626      715      +89     
  Lines       32302    36814    +4512     
==========================================
+ Hits        22470    25308    +2838     
- Misses       8588    10184    +1596     
- Partials     1244     1322      +78     ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
  | 
    
20a1b03    to
    acdf77c      
    Compare
  
    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.
Ok, let's do a demo tomorrow 😂
| 
           I note that we have a lot of communality between this PR and what would be required to support oauth "3 legged" in other "classic" PSP.  | 
    
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.
didnt' mean to answer as another review, oops
a55b484    to
    4c896ce      
    Compare
  
    0b52402    to
    053bb80      
    Compare
  
    
          
 Might be nice if you tried to increase the code coverage for the new code you added though 🙂  | 
    
e0e1a04    to
    76ee5e5      
    Compare
  
    
No description provided.