-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add airdrop example #18
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see my comments and let me know if something is unclear.
airdrop/readme.md
Outdated
|
||
![ccd-transaction-1.png](docs/ccd-transaction-init.png) | ||
|
||
If you confirm it, you will see a loader with a link to concordium explorer where you can get more information about the transaction. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the developer documentation, we refer to the the below picture as the concordium dashboard while the concordium explorer is CCDScan
https://ccdscan.io/. This is the convention for the developer documentation also I know that on the general public webpage we are not consistent with this:
https://concordium.com/block-explorers/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm a little confused as to what exactly needs to be fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The above picture is from the Concordium dashboard
. Your front end links to the dashboard. Since your documentation talks about Concordium explorer
, it would be good to link to https://ccdscan.io/{txHash}
in the front end and update the pictures or vice versa. I suggest using the Concordium explorer
so.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added link to concordium explorer [concordium explorer](https://dashboard.testnet.concordium.com/lookup/{txhash})
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It could also be at this addresshttps://testnet.ccdscan.io/?dcount=1&dentity=transaction&dhash={txhash}
, but dashboard.testnet.concordium.com
seems more informative to me
Co-authored-by: Doris Benda <[email protected]>
Co-authored-by: Doris Benda <[email protected]>
Co-authored-by: Doris Benda <[email protected]>
Co-authored-by: Doris Benda <[email protected]>
Co-authored-by: Doris Benda <[email protected]>
Co-authored-by: Doris Benda <[email protected]>
When I was testing the program I was asked to make an excel table with all the tests. Code fragments were written in a separate branch, which automatically added transactions to the table (something like logging) and I only had to add a description of the test. Is it necessary to add this code here? |
Purpose
Example dApp for showcasing how to integrate with a smart contract on the Concordium blockchain.
The project was bootstrapped with Vite.
Changes
Adding a project to the repository
Checklist
hard-to-understand areas.