Skip to content

feat(credentials)!: Account Update functions#435

Draft
JR-Morgan wants to merge 1 commit intomainfrom
jrm/account-update
Draft

feat(credentials)!: Account Update functions#435
JR-Morgan wants to merge 1 commit intomainfrom
jrm/account-update

Conversation

@JR-Morgan
Copy link
Member

@JR-Morgan JR-Morgan commented Jan 29, 2026

V3 never updates the account credentials of accounts in the sqlite db
So user and server information remains stale.

I've fixed the functions in the SDK, and also added a way to upgrade only a single account (before you had to upgrade all accounts)
Not sure yet how/where we'll use these functions.

@JR-Morgan JR-Morgan changed the title feat(credentials): Account Update functions feat(credentials)!: Account Update functions Jan 29, 2026
@codecov
Copy link

codecov bot commented Jan 29, 2026

Codecov Report

❌ Patch coverage is 1.13636% with 87 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.12%. Comparing base (00a6619) to head (e2429d4).

Files with missing lines Patch % Lines
src/Speckle.Sdk/Credentials/AccountManager.cs 2.12% 46 Missing ⚠️
...sts.Integration/Credentials/AccountManagerTests.cs 0.00% 36 Missing ⚠️
...sts.Integration/Credentials/AccountFactoryTests.cs 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #435      +/-   ##
==========================================
- Coverage   64.14%   55.12%   -9.02%     
==========================================
  Files         289      330      +41     
  Lines       11835    13771    +1936     
  Branches     1137     1187      +50     
==========================================
  Hits         7591     7591              
- Misses       3928     5865    +1937     
+ Partials      316      315       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant