velodrome-finance / sugar

Sugar makes Velodrome devs life sweeter 🍭
Other
47 stars 26 forks source link

Add governance amount and delegate ID to VeSugar #32

Closed ethzoomer closed 1 year ago

ethzoomer commented 1 year ago

@airtoonricardo If you don't mind pls double check this to confirm if it's the proper delegate implementation for what we discussed earlier

airtoonricardo commented 1 year ago

@airtoonricardo If you don't mind pls double check this to confirm if it's the proper delegate implementation for what we discussed earlier

My apologies, I thought the intended use case would be to have the whole checkpoint's information in order to be able to display delegated balances and other information.

Since we are only fetching who the veNFT is delegating to, we should use the self.ve.delegates(_id) instead, as Stas has mentioned.