Skip to content

Feat/v1#36

Merged
jmdelcarmen merged 24 commits into
masterfrom
feat/v1
Jul 31, 2024
Merged

Feat/v1#36
jmdelcarmen merged 24 commits into
masterfrom
feat/v1

Conversation

@ossiggy

@ossiggy ossiggy commented Jun 3, 2024

Copy link
Copy Markdown
Contributor
  • Adds all v2 methods, along with tests
  • Updates Readme
  • Bumps to v1.0.0

@ossiggy ossiggy requested review from jmdelcarmen and mustafalii June 3, 2024 17:07
@ossiggy ossiggy requested review from stephenluc June 4, 2024 15:03
@radoshi

radoshi commented Jul 31, 2024

Copy link
Copy Markdown

Hi @ossiggy what's the status of committing this into the repo? There are some things here that we'd like to play around with (eg. CreditScore) that are not available in the released version of the lib. Thanks!

@dwilliams27 dwilliams27 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall lgtm



class PlaidBalance(TypedDict):
available: Optional[int]

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor, but the spacing seems off on some of the classes in this file

usage_pattern: Optional[AccountLiabilityCreditCardUsageTypesLiterals]


class AccountLiabilityCollection(AccountLiabilityBase):

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this used anywhere?

@sbilalh sbilalh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could use some linting but otherwise fine

@jmdelcarmen jmdelcarmen merged commit 28027dd into master Jul 31, 2024
@ossiggy

ossiggy commented Jul 31, 2024

Copy link
Copy Markdown
Contributor Author

Hi @ossiggy what's the status of committing this into the repo? There are some things here that we'd like to play around with (eg. CreditScore) that are not available in the released version of the lib. Thanks!

@radoshi this has been merged

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.

5 participants