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

✨ swap page landing #11163

Merged
merged 55 commits into from
Dec 9, 2024
Merged

✨ swap page landing #11163

merged 55 commits into from
Dec 9, 2024

Conversation

roiLeo
Copy link
Contributor

@roiLeo roiLeo commented Oct 23, 2024

Screenshot 2024-10-23 at 11-52-32 Swap NFTs One Stop Shop for Polkadot NFTs

@roiLeo roiLeo requested a review from a team as a code owner October 23, 2024 10:05
@roiLeo roiLeo requested review from vikiival and hassnian and removed request for a team October 23, 2024 10:05
Copy link

netlify bot commented Oct 23, 2024

Deploy Preview for koda-canary ready!

Name Link
🔨 Latest commit ee70ff5
🔍 Latest deploy log https://app.netlify.com/sites/koda-canary/deploys/67551ab734746c000801e467
😎 Deploy Preview https://deploy-preview-11163--koda-canary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@hassnian hassnian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!

@roiLeo I see that the pr does not close the issue, do you want to finish it ? otherwise I can take over.

can you

  • use translation keys for all words
  • limit this page to only assethub inside redirect.global using getPermissionRouteCondition -> permission.config.ts
  • if you are not logged this should be a button (I guess like the transfer page)
    CleanShot 2024-10-24 at 08 57 12@2x

CleanShot 2024-10-24 at 08 57 57@2x

@roiLeo roiLeo marked this pull request as draft October 24, 2024 06:27
@roiLeo
Copy link
Contributor Author

roiLeo commented Oct 24, 2024

I see that the pr does not close the issue, do you want to finish it ? otherwise I can take over.

Yea it's WIP, feel free to take it if you need to ship it faster

use translation keys for all words

yes, that would be ideal

limit this page to only assethub inside redirect.global using getPermissionRouteCondition -> permission.config.ts
if you are not logged this should be a button (I guess like the transfer page)

noted! ty

@roiLeo
Copy link
Contributor Author

roiLeo commented Oct 24, 2024

@hassnian any way to split the work? so we don't erase each other stuff
like someone handle the logic, any other step or maybe you want full control?

@hassnian
Copy link
Contributor

@hassnian any way to split the work? so we don't erase each other stuff like someone handle the logic, any other step or maybe you want full control?

yeah it's gonna be hard since its a lot of new code, I'll let you finish it. I can work on something else in the meantime

@vikiival
Copy link
Member

Well how swap works:

  • you take your NFT
  • you select nft you want
  • call same call as offer :P

So my recommendation is that one can start with item detail:

  • create button, or add swap for another in the more select

The Swap page (cc @exezbcz):
On one side is your nft
on the other is either desired nft or considered collection

It means if desired on the indexer is null you want nft from particular collection

Future
You can additionally add surcharge which https://github.com/kodadot/stick/blob/39ac052e60cc0b9c03ca1d3e23cb6c3f869e5822/schema.graphql#L244

You can either send or receive

@vikiival
Copy link
Member

However I like also your idea to invite a "conter-party" so you can easily look for "his/hers" nfts

components/items/ItemsGrid/ItemsGridImage.vue Outdated Show resolved Hide resolved
components/swap/landing.vue Outdated Show resolved Hide resolved
@hassnian hassnian requested review from preschian and removed request for hassnian December 2, 2024 09:27
@hassnian
Copy link
Contributor

hassnian commented Dec 2, 2024

I'm removing myslef and asking @preschian for a review since there's a lot of my code on this pr

Copy link
Member

@preschian preschian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

composables/transaction/transactionCreateSwap.ts Outdated Show resolved Hide resolved
hassnian and others added 2 commits December 3, 2024 17:56
Co-authored-by: Preschian Febryantara <preschian@users.noreply.github.com>
@hassnian hassnian added the S-code-lgtm-✅ code review guild has reviewed this PR and it's code is approved label Dec 5, 2024
# Conflicts:
#	utils/config/permission.config.ts
Copy link

sonarqubecloud bot commented Dec 8, 2024

@hassnian
Copy link
Contributor

hassnian commented Dec 9, 2024

@vikiival ready to merge

@vikiival
Copy link
Member

vikiival commented Dec 9, 2024

I Can confirm that implementation works.
tried 1-1

Small improvements:

  • There is no way (or I havent found) how to get to the /swap page
  • after success app should take you to your profile / swaps / outgoing - currently it redirects you to /swaps

@hassnian
Copy link
Contributor

hassnian commented Dec 9, 2024

I Can confirm that implementation works. tried 1-1

Small improvements:

  • There is no way (or I havent found) how to get to the /swap page

yep also noticed that #11216

  • after success app should take you to your profile / swaps / outgoing - currently it redirects you to /swaps

sure, let's do it in a follow up since the code has been tested and also reviewed

@vikiival vikiival added this pull request to the merge queue Dec 9, 2024
Merged via the queue into kodadot:main with commit 9c6aacf Dec 9, 2024
22 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-atomic-swap S-code-lgtm-✅ code review guild has reviewed this PR and it's code is approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create Swap Page
5 participants