aydenp / bank Goto Github PK

Probably due to long-running generation function receiving data after range has been changed

If you can make it able to be used on jailbroken devices it would be good because most banks block jailbroken devices

Currently when we refresh, if accounts change the view controllers will be re-created and set. This is good, except for when minor details that we can represent in the view controller change, such as account name or balance. When this happens, we should fire a different notification that results in the new account information getting set on the view controller, without creating a new one.

Currently, the app is very reliant on SessionDataStorage. This is a huge fault as I try to move everything to a separate framework, BankKit, as it doesn't seem right for the framework to be housing this data in a singleton. Rather, the manager should find a way to create individual sessions, which can store a Plaid item container, which will perform similarly to how SessionDataStorage does right now, except tied to a given Item. This would require a substantial rewrite.

Eventually, when we want to add multiple bank logins per app, it'll also come in handy, as it doesn't make sense to have one for all.

The app currently has no icon

Today widget for fast balance and last transaction viewing

By tapping on the line chart, you should be able to view the balance for a certain date. The balance label should change to that, the transactions view should fade out a bit, and the date should appear above the label. You should be able to escape this view by tapping outside of the chart.

I have Plaid API and everything but can I run this for sandbox, and also how do i put the plaid.example.plist file into plaid.plist? I keep encountering this error and that is why the code doesn't run.

Since every page has its own page control, it looks kinda bad when scrolling. If both pages are at the top, it should move from one page to the other as the user scrolls.

When I run the app in swift, all the functions are working. But the location including (address, city, zip code, etc.) show the value is null. I wonder if there is no location data or I did wrong.

Currently the line chart only shows 30 days back. It should support increments of 1 week, 2 weeks, 1 month, 3 months, 6 months, 1 year, and all transactions ever.

Allow setting this via environment key in Plaid.plist

Apple Watch app with recent transactions and balance for all accounts

To support more than one account, the app lets you swipe horizontally to see your other accounts. This swipe gesture still works if there is only one account, and just seems weird having the elastic scroll without any other view controllers, so we should find a way to disable it when there is only one account there.

Currently, when you scroll down, content goes right under the status bar. In the future, this shouldn't happen.

Ideally, the status bar will have a view underneath it with the same background gradient the transaction view has, and a hairline that appears as soon as content goes underneath it.

The transaction line chart data generator can probably be sped up by only getting the balance on days before, of, and after transactions take place, rather than every single day.

Two issues were discovered when testing that prohibit the use of the app beyond the 'Link your Bank Account' button.

Issue 1 - After inputting API Keys I received an error message in the debug window advising me of missing parameters, specifically the inclusion of "country_codes". Looking at the API documentation and code I pinpointed it to you "institutions/get" call in the PeriodicFetchDataStorage Class -> public let institutions variable. This seems to be missing an array of country_codes. To fix this issue I just added "country_codes": ["US"] to string. Code attached.

PeriodicFetchDataStorage (updated).txt

Issue 2 - After fixing the above issue I came across another related to the use of the public key. Tracing it to your PlaidManager class it seems the use of PublicKey is no longer supported by Plaid and has since been depreciated. The API now uses Public_Token. Unable to pinpoint fix for this.

We have a lot of information on transactions that occur, but show so little of it in the table. Add a transaction view controller accessed by tapping on a transaction to show all the information we have.

On first account pull, transaction data won't be complete. We should keep checking periodically until it is (if checking is possible).

Using transaction history, add a line graph showing account balance over time to fill the whitespace under the balance label.

Support showing PlaidManager initialization errors in StatusViewController

Currently the page control is at the bottom of the screen. It would probably look better underneath the balance label, and fix the layout issues causing it to appear under the notch.

Hi there,

I am wondering if it's possible to port this app over to Android or if this app can only work with iOS devices.

Thanks!