Uh oh!
There was an error while loading. Please reload this page.
fix(credentials): credential dependent endpoints - #3309
Conversation
The latest updates on your projects. Learn more about Vercel for GitHub. |
Greptile SummaryThis PR fixes dependent endpoints (Gmail, Outlook, OneDrive, SharePoint, Wealthbox, Microsoft Planner, Vertex AI) to properly resolve credential IDs using the new Key Changes:
Minor Change:
The implementation is consistent across all files, with proper null checks, error logging, and early returns when credential resolution fails. Confidence Score: 5/5
Important Files Changed
Last reviewed commit: e74a04a |
Uh oh!
There was an error while loading. Please reload this page.
icecrasher321
commented
Feb 23, 2026
bugbot run |
Uh oh!
There was an error while loading. Please reload this page.
icecrasher321
commented
Feb 23, 2026
bugbot run |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
icecrasher321
commented
Feb 23, 2026
bugbot run |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
* fix(dependent): credential dependent endpoints * fix tests * fix route to not block ws creds" * remove faulty auth checks: * prevent unintended cascade by depends on during migration * address bugbot comments
Summary
Dependent endpoints like Gmail Fetch label / Webhook external subscriptions should use new system helper to resolve credential id.
Type of Change
Testing
Tested manually
Checklist