ArboreumDev / credit-union-frontend

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

adding a transaction table #65

Closed djudjuu closed 3 years ago

djudjuu commented 4 years ago

closes #44

context for design decisions

the PK for the tx-table is an int...and not a uuid... I thought it would be nice if we could very easily see which transaction came first and it would be handy to impose an ordering on the txs a user does....a little later I realized that the same could be achieved with created_at...I dont see a downside of having the PK be a tx though....

Open question:

vercel[bot] commented 4 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/arboreum/frontend/r5q2ynowj
✅ Preview: https://frontend-git-dju-transactions.arboreum.vercel.app

[update in progress for 5841b2c]

gparuthi commented 4 years ago

Looks good to merge once we resolve the id column issue