feat(TBF-163): remove Teller Connect and live API support - #178
Conversation
Keep legacy Teller connections displayable with sync sunset errors and unchanged disconnect; demo mode now seeds SimpleFIN. Co-authored-by: Cursor <cursoragent@cursor.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: edad3a4b27
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| let provider = connection.provider.clone(); | ||
| if !state.config.is_financial_provider_enabled(&provider) { | ||
| if provider != "teller" && !state.config.is_financial_provider_enabled(&provider) { |
There was a problem hiding this comment.
Avoid consuming quota for sunset Teller syncs
When the connection is a legacy Teller row, this branch lets the request continue into try_consume_sync_quota before TellerSyncDispatcher returns TELLER_NO_LONGER_SUPPORTED. Because the UI still exposes “Sync now” for Teller connections, every unsupported Teller attempt burns the user's daily sync quota (and after the limit they see RATE_LIMITED instead of the sunset error), which can also block a real sync later that day after they disconnect/relink. Return the Teller sunset response before consuming quota.
Useful? React with 👍 / 👎.
|
🎉 This PR is included in version 7.1.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
TELLER_NO_LONGER_SUPPORTED; disconnect unchangedTest plan
bun run backend:ciand focused frontend tests / typecheckMade with Cursor