Magickbase / ckb-explorer-public-issues

CKB Explorer Issues
https://explorer.nervos.org/
3 stars 2 forks source link

Organize a PRD #60

Closed Keith-CY closed 1 year ago

Keith-CY commented 2 years ago

Acceptance team is going to have a regression test on CKB explorer but blocked because the original feature descriptions are missing, so we have to re-organize one to functions of CKB explorer clear.

Keith-CY commented 2 years ago

General Components:

Keith-CY commented 2 years ago

Homepage: https://explorer.nervos.org/

Keith-CY commented 2 years ago

Block List Page: https://explorer.nervos.org/block/list

Keith-CY commented 2 years ago

Transaction List Page: https://explorer.nervos.org/transaction/list

Keith-CY commented 2 years ago

Block Page: /block/{number | hash}

Keith-CY commented 2 years ago

Transaction Page: /transaction/{hash}

Keith-CY commented 2 years ago

Address Page: /address/${hash}

Keith-CY commented 2 years ago

Sudt List Page: /tokens

Keith-CY commented 2 years ago

Sudt Info Page: /sudt/{hash}

Keith-CY commented 2 years ago

NFT Collection List Page: /nft-collections

Keith-CY commented 2 years ago

NFT Collection Page: /collections/{hash | cota id}

Keith-CY commented 2 years ago

NFT Token Item Page: /nft-info/{collection hash | cota id}/{token item id}

Keith-CY commented 2 years ago

Script List Page: /scripts

Keith-CY commented 2 years ago

Charts Page: /charts or /charts/{chart name}

Keith-CY commented 2 years ago

Other pages

Keith-CY commented 1 year ago

I've created a PR(https://github.com/Magickbase/ckb-explorer-public-issues/pull/73) to propose the PRD in the repo, any questions could be commented in the PR and I'll add more details.

Dawn-githup commented 1 year ago

@Keith-CY Does GraphQL need to be tested

Keith-CY commented 1 year ago

@Keith-CY Does GraphQL need to be tested

This issue is for ckb explorer but ckb explorer doesn't use graphql.

If you're asking graphql used in gwscan then yes, graphql in gwscan should be fully tested.

Recently we found the test cases are not enough https://github.com/Magickbase/godwoken_explorer/issues/1034#issuecomment-1268806923 although they have been updated(https://github.com/Magickbase/godwoken_explorer/issues/589). We're going to add more.

If you're asking APIs of ckb explorer backend, now it's not a public service so tests of APIs are not so important as tests of UI functions

Dawn-githup commented 1 year ago

@Keith-CY Does GraphQL need to be tested

This issue is for ckb explorer but ckb explorer doesn't use graphql.

If you're asking graphql used in gwscan then yes, graphql in gwscan should be fully tested.

Recently we found the test cases are not enough Magickbase/godwoken_explorer#1034 (comment) although they have been updated(Magickbase/godwoken_explorer#589). We're going to add more.

If you're asking APIs of ckb explorer backend, now it's not a public service so tests of APIs are not so important as tests of UI functions

ok i see ,i was asking about ckb explorer