steven
/
ynab-portfolio-monitor
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Define common interface for account and security providers #1

New Issue
Closed
opened 2023-11-12 19:19:52 +00:00 by steven · 2 comments
steven commented 2023-11-12 19:19:52 +00:00
Owner
Copy Link

Every account provider (eg questrade, crypto currency wallet, computershare, etc) should have a common set of READ only based operations to determine the account balance or securities contained within the account.

This should make adding integrations much more streamlined.

Every account provider (eg questrade, crypto currency wallet, computershare, etc) should have a common set of READ only based operations to determine the account balance or securities contained within the account. This should make adding integrations much more streamlined.
steven self-assigned this 2023-11-12 19:19:52 +00:00
steven added this to the Minimum Viable Product milestone 2023-11-12 19:22:15 +00:00
steven added the
enhancement
 label 2023-11-12 19:22:18 +00:00
steven commented 2023-11-13 00:28:46 +00:00
Author
Owner
Copy Link

Each provider should have a Configure() method which attempts to configure the provider using environment variables. If the configuration is successful, then the provider is used otherwise it is not used. The Configure method should contain some sanity checking on minimum viable configuration and if the threshold is not met then the configuration will not be successful.

Each provider should also have a GetBalance() method provides a total account balance for that provider with the implementation details abstracted away.

Each provider should have a Configure() method which attempts to configure the provider using environment variables. If the configuration is successful, then the provider is used otherwise it is not used. The Configure method should contain some sanity checking on minimum viable configuration and if the threshold is not met then the configuration will not be successful. Each provider should also have a GetBalance() method provides a total account balance for that provider with the implementation details abstracted away.
steven commented 2023-11-13 22:18:50 +00:00
Author
Owner
Copy Link

Added in 7284545571

Added in 72845455711e7e5eb4f3fc11fcb3e33c0d508409
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
main
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No Label
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Minimum Viable Product
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
steven
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: steven/ynab-portfolio-monitor#1
No description provided.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?