ArboreumDev / credit-union-frontend

Frontend in Next.js + Typescript + GraphQL
https://frontend-two-sandy.vercel.app
1 stars 0 forks source link

add decentro client and simple test #126

Closed gparuthi closed 4 years ago

gparuthi commented 4 years ago

Design doc moved to notion

djudjuu commented 4 years ago

@gparuthi

Questions

Remove balance column from user table.

i am not sure, I like this. We should know the user balance even if decentro is offline/unavailable. Or is this someting we should do in a later step? I need to get a better look at the decentro api

make sure the amounts on our side and wallet are always in sync

so, if not in the user-table, where would be our side?

Add UserModel with balance getter. LenderModel and BorrowerModel can inherit from this class.

I dont see why these need to be different atm. I thought none of them are using decentro direclty, so all they see is their bank account and their account on our site.... Can u elaborate this point a bit?

Borrower-Repayment and Lender-investments will be initiated by the user

let's break this down a bit more.

UPI payment app like Google Pay.

i know nothing about this.Looks like there is testing tooling here: https://developer.paytm.com/docs/testing-integration/ but I jus twant to point out, that this might be a bit longer for me to get into..

braindump

action items:

let's breakdown how we can start working on this:

gparuthi commented 4 years ago

Continuing the conversation in notion. I added information that should answer your questions. Let me know if I missed anything.