Skip to content

Add CC support#272

Merged
Nef10 merged 17 commits intomainfrom
cc
Nov 11, 2025
Merged

Add CC support#272
Nef10 merged 17 commits intomainfrom
cc

Conversation

@Nef10
Copy link
Owner

@Nef10 Nef10 commented Nov 10, 2025

  • Add GraphQL requests for the CC, as the transactions are not available via REST API
  • Support pagination - as we have this now it will be required to specify a start date
  • SDo a second request for the CreditCardActivity entries on top of the FetchActivityFeedItems, as only the first one includes foreign transaction currency and amount
  • Re-organize code, move supporting parts to new Transaction file

@Nef10 Nef10 added feature New feature major This change should bump the major version of the next release labels Nov 10, 2025
@github-actions
Copy link

Minimum coverage required: 96%


Coverage after merging cc into main will be

97.12%

Coverage Report
FileStmtsFuncsLinesUncovered Lines
/Users/runner/work/WealthsimpleDownloader/WealthsimpleDownloader/Sources/Wealthsimple
   Token.swift97.29%95.24%97.50%117–118, 137–138
   Transaction.swift88.06%42.86%93.33%
   URLConfiguration.swift100%100%100%
   WealthsimpleAccount.swift89.01%63.64%92.50%114–115
   WealthsimpleAsset.swift81.82%33.33%89.47%
   WealthsimpleDownloader.swift99.22%100%99.14%60
   WealthsimplePosition.swift91.53%66.67%94.34%117–118
   WealthsimpleTransaction.swift98.10%93.55%98.42%155, 247, 282, 345–346
/Users/runner/work/WealthsimpleDownloader/WealthsimpleDownloader/Sources/Wealthsimple/Extensions
   String+CamelCase.swift100%100%100%

@Nef10 Nef10 merged commit bab5b50 into main Nov 11, 2025
3 checks passed
@Nef10 Nef10 deleted the cc branch November 11, 2025 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature major This change should bump the major version of the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant