regen-network / rnd-dev-team

RND Dev Team issue management (DEPRECATED)
0 stars 0 forks source link

Add credit batch table to project page #937

Closed blushi closed 2 years ago

blushi commented 2 years ago

https://www.figma.com/file/seUiQM2SPt78KJiDL0Ea8m/Basic-Ecocredits-Ledger-View?node-id=275%3A40553

This was partially already implemented as part of https://github.com/regen-network/regen-web/pull/858 through ProjectCreditBatchesTable.tsx but we need to fetch batches based on the regen:vcsProjectId metadata field similarly to what's been done in https://github.com/regen-network/regen-web/pull/943

We should also update existing project metadata (staging and prod) and references of this in the code to use "regen:vcsProjectId": 123 instead of "regen:vcsProjectId": { "@type": "xsd:unsignedInt", "@value": 123 } Ref: https://github.com/regen-network/regen-registry-ops/pull/7#issuecomment-1130562234


For Admin Use

blushi commented 2 years ago

Hey team! Please add your planning poker estimate with ZenHub @haveanicedavid @mhagel @victordelval

blushi commented 2 years ago

@erikalogie this is r4r on https://deploy-preview-959--regen-registry.netlify.app/projects/kasigau

erikalogie commented 2 years ago

This looks great. Should we add the tx hash to the left-hand column? https://www.figma.com/file/seUiQM2SPt78KJiDL0Ea8m/Basic-Ecocredits-Ledger-View?node-id=1585%3A93706

On Tue, May 24, 2022 at 2:40 PM Marie Gauthier @.***> wrote:

@erikalogie https://github.com/erikalogie this is r4r on https://deploy-preview-959--regen-registry.netlify.app/projects/kasigau

— Reply to this email directly, view it on GitHub https://github.com/regen-network/regen-registry/issues/937#issuecomment-1135869262, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACARPTMVIG4JPYQNNFDHHR3VLTE2LANCNFSM5VIG57YQ . You are receiving this because you were mentioned.Message ID: @.***>

blushi commented 2 years ago

This looks great. Should we add the tx hash to the left-hand column?

this will get added through @mhagel work once https://github.com/regen-network/regen-web/pull/957 gets merged since the same underlying component is being used.

erikalogie commented 2 years ago

Great!

On Wed, May 25, 2022 at 3:11 PM Marie Gauthier @.***> wrote:

This looks great. Should we add the tx hash to the left-hand column?

this will get added through @mhagel https://github.com/mhagel work once regen-network/regen-web#957 https://github.com/regen-network/regen-web/pull/957 gets merged

— Reply to this email directly, view it on GitHub https://github.com/regen-network/regen-registry/issues/937#issuecomment-1137220463, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACARPTPQQ3WCP5PUJHQL2QTVLYRJXANCNFSM5VIG57YQ . You are receiving this because you were mentioned.Message ID: @.***>