Skip to content
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

Review: Minting flow #389

Open
jdsika opened this issue Dec 4, 2024 · 2 comments
Open

Review: Minting flow #389

jdsika opened this issue Dec 4, 2024 · 2 comments
Assignees

Comments

@jdsika
Copy link
Contributor

jdsika commented Dec 4, 2024

Things I did observe while uploading

  1. Uploading asset.zip with strange renaming
    grafik
    Suggestion: directly rename to the CID. We only need one defined way of renaming and this needs to happen anyway.

  2. Auto update processing Auto update processing status #384

  3. Where are the assets stored? I know that I did login a few days ago and uploaded the asset for test. I was expecting it to still exists. Are uploaded assets still in the list when I log out?

  4. Add a delete uploaded assets button Asset uploader: delete button #388

  5. The asset name is changing again after the upload -> confusing:
    grafik

    -> Added to Rework table: "Add Assets" History #395

  6. First minting attempt failed. Not sure why. I will mint again. Please check if on a failed mint the data was already uploaded to IPFS and if that data is reused when attempting to mint again.
    6.1) Minting was apprently successful but the UI did not catch that correctly:
    grafik

  7. Beacon shows "successful mint" now

  8. Status "minted" is shown now. Will this serve as the complete history? Maybe that is in fact useful?
    grafik
    Suggestion: Rename "Assets that are uploaded and ready for minting." to indicate it is the full transaction history.

@jdsika
Copy link
Contributor Author

jdsika commented Dec 4, 2024

Pinata Groups:
Please use correct CAIP-10 DID nomenclature: did:pkh:tezos:chain_id:address OR just the address

Shouldn't we use individual names for thr token data to make it easier to identify instead of token_info.json

@jdsika
Copy link
Contributor Author

jdsika commented Dec 6, 2024

Decision to point 5

  • The dashboard contains the assets overview that contains all assets that have been minted
  • under "add assets" a user can see the history of actions the user has conducted including the status
  • we should add more columns and make this explicit with a proper headline

name | CID | time | status

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants