added validator section #94
Conversation
|
Pls fmt and make the test to pass @PoulavBhowmick03 |
|
@ONEONUORA can you check now |
|
Hi @PoulavBhowmick03 You should run |
|
Hi @PoulavBhowmick03 there's already a validator profiles table in the migrations script, not sure why you're creating another one; Are you still working on this? |
Abeeujah
left a comment
Abeeujah
left a comment
There was a problem hiding this comment.
Based on the rest of the modelling being built on the schema you provided for yourself, and no query caching to have these validated, I don't think there's much to be reviewed here though, we'd progress after the Schemas have been fixed
|
|
||
| -- index for fast wallet lookups | ||
| CREATE INDEX IF NOT EXISTS idx_validators_wallet | ||
| ON validators(wallet_address); |
There was a problem hiding this comment.
Hi @PoulavBhowmick03 after going through the setup migration, anything particularly wrong with the validators table/schema in the migration script that spurred you to go ahead and create yours?
3 participants
fixes #67