kodadot / nft-gallery

Generative Art Marketplace
https://kodadot.xyz
MIT License
611 stars 351 forks source link

Migration UI - 1 - Home page #7560

Closed exezbcz closed 7 months ago

exezbcz commented 8 months ago

part of a bigger issue:

Home page

Top part

image

source and destination

Faq

Selection

Ready for Migration

Waiting for your action

image

migration not possible

yangwao commented 8 months ago

Hey @kodadot/internal-dev if someone would pick this one to start?

preschian commented 8 months ago

I will check this one

preschian commented 8 months ago

Actually, I'm still not familiar with the migration script. I will try to finish the UI first with dummy data and integrate it with real data later

kodabot commented 8 months ago

ASSIGNED - @preschian πŸ”’ LOCKED -> Saturday, October 14th 2023, 22:48:15 UTC -> 36 hours

vikiival commented 8 months ago

Actually, I'm still not familiar with the migration script. I will try to finish the UI first with dummy data and integrate it with real data later

I think let's start with UI.

kodabot commented 8 months ago

ASSIGNMENT EXPIRED - @preschian has been unassigned.

preschian commented 8 months ago

@vikiival how to check if the collection is ready to migrate, waiting, or not supported?

vikiival commented 8 months ago

Any collection issuer and current owner of is possible to be migrated On both KSM and RMRK

preschian commented 8 months ago

image

@vikiival

  1. How do we check if the collection has already been migrated?
  2. Does it mean migration was impossible if the collection owner had yet to migrate? Is that correct?
vikiival commented 8 months ago

How do we check if the collection has already been migrated?

Good Q. Will add it to the API. The full desc is here https://github.com/kodadot/nft-gallery/issues/7562#issuecomment-1772705044

Therefore you can ask api if collection was migrated or not

Does it mean migration was impossible if the collection owner had yet to migrate? Is that correct?

Sorry, I do not understand your question. Can you please rephrase your request?

preschian commented 8 months ago

Good Q. Will add it to the API. The full desc is here #7562 (comment)

πŸ‘πŸ‘

Does it mean migration was impossible if the collection owner had yet to migrate? Is that correct?

Sorry, I do not understand your question. Can you please rephrase your request?

oops, sorry, I mean, if the collection owner has not yet migrated. for this section:

image

vikiival commented 8 months ago

oops, sorry, I mean, if the collection owner has not yet migrated. for this section:

Like You can migrate anything

API has call for all you your collection that you have migrated

GET /relocations/issuers/${accountId}

returns a list

Currently I am unable to deploy waifu worker

Screenshot 2023-10-25 at 17 13 26

preschian commented 8 months ago

API has call for all you your collection that you have migrated

GET /relocations/issuers/${accountId}

returns a list

Currently I am unable to deploy waifu worker

lmk if the endpoint deployed πŸ‘ I will open followup PR

Like You can migrate anything

@vikiival @exezbcz, should we update the "Migration Not Possible" design?

vikiival commented 8 months ago

mk if the endpoint deployed πŸ‘ I will open followup PR

published the waifu worker

There are no data yet full desc was updated in https://github.com/kodadot/nft-gallery/issues/7562#issuecomment-1772705044

exezbcz commented 8 months ago

should we update the "Migration Not Possible" design?

yeah we can remove it :D

vikiival commented 7 months ago

cc @preschian any blocker here?

preschian commented 7 months ago

any blocker here?

hi hi, I need help with this one #8206

regarding this https://github.com/kodadot/nft-gallery/pull/8201#issuecomment-1824594927, iiuc @exezbcz will have some updated version

exezbcz commented 7 months ago

should we update the "Migration Not Possible" design?

yeah we can remove it :D

I guess this, the waiting for your action design is still valid

preschian commented 7 months ago

should we update the "Migration Not Possible" design?

yeah we can remove it :D

I guess this, the waiting for your action design is still valid

@exezbcz hhmm, I don't understand. Could you elaborate more? In PR #8201, that section was removed. need to adjust this one, right? (waiting for your action section) https://github.com/kodadot/nft-gallery/pull/8201#issuecomment-1824594927

especially I need help with these parts:

exezbcz commented 7 months ago
  • for the bottom part, I will probably change the text a bit -> what is the text?

this will be removed, the part is not needed

  • what/how is the correct one?

correct one is that it shows what is described in the issue

  • here are items from collections you don't own, but the collection was migrated and initiated - giving you the option to migrate as well.
  • Displayed here are individual items that you own from collections that have been migrated. Ensure you're up-to-date by migrating these pieces right away. Please note you can only start the migration once the collection owner initiates it.

does that make sense? cc @preschian

preschian commented 7 months ago

@exezbcz I probably misunderstood your requirements. Here are some updates in #8280

"Budget Spaceships" -> I have one item from this collection. But, the owner still has not migrated the collection yet -> there are no buttons on the card. "AI Cyberpunks" -> I have one item, and the owner already migrated that to ahk -> there is a button to migrate my items.

Is that correct?

image

preschian commented 7 months ago

@exezbcz I mocked the address locally with your address. It should show up like this

image

preschian commented 7 months ago

hmmm, wait, maybe the titles between "Ready for Migration" and "Waiting For Your Action" are confusing. We need to change the words, I guess? Because the user can't do anything unless the collection owner migrates the collection

exezbcz commented 7 months ago

Budget Spaceships" -> I have one item from this collection. But, the owner still has not migrated the collection yet -> there are no buttons on the card.

the card should not be visible, its showed only after its possible to follow the migration, that is why its "waiting for your action"

It should show up like this

yeah but without the waiting for your action section - owner of the collections did not migrate the collection yet

preschian commented 7 months ago

the card should not be visible

aahh, I see, hide the card completely instead. will update that

exezbcz commented 7 months ago

@preschian correct, plus when there is no card, the section is hidden