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

fix multi recipient royalties on Details Tab #7013

Merged
merged 4 commits into from
Aug 30, 2023

Conversation

niklasp
Copy link
Contributor

@niklasp niklasp commented Aug 28, 2023

PR Type

  • Bugfix
  • Feature
  • Refactoring

Context

Did your issue had any of the "$" label on it?

Copilot Summary

🤖 Generated by Copilot at 7483263

Display and style NFT transfer recipients in GalleryItemDescription.vue. Use a computed property to format the data and a template to render it.

🤖 Generated by Copilot at 7483263

Oh we're the coders of the NFT sea
And we display the data as it should be
With a computed property and a template so fine
And some custom CSS classes to make it shine

Description

backward compatible version of multiroyalties according to the design guide given

http://localhost:9090/ahk/gallery/101-3962765076 => multi royalty) in array format [[addr0, perc0], [addr1, perc1], ...]
Bildschirmfoto 2023-08-28 um 19 14 29

http://localhost:9090/ahk/gallery/101-3163187696 => single royalty in array format [[addr0,perc0]]
Bildschirmfoto 2023-08-28 um 19 14 33

Bildschirmfoto 2023-08-28 um 19 14 36

http://localhost:9090/ksm/gallery/19381324-BC4ECCF7AB22138D66-SMS-SOL_SMS_22_KSM-00000001 (old royalty, just address)

@niklasp niklasp requested a review from a team as a code owner August 28, 2023 17:15
@niklasp niklasp requested review from preschian and roiLeo and removed request for a team August 28, 2023 17:15
@netlify
Copy link

netlify bot commented Aug 28, 2023

Deploy Preview for koda-canary ready!

Name Link
🔨 Latest commit 71d70fb
🔍 Latest deploy log https://app.netlify.com/sites/koda-canary/deploys/64ee7498e5f1d00008273396
😎 Deploy Preview https://deploy-preview-7013--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.

@reviewpad
Copy link
Contributor

reviewpad bot commented Aug 28, 2023

AI-Generated Summary: This pull request contains changes that fix issue #6970. The modifications are made to the 'GalleryItemDescription.vue' file. Changes include adding a condition to check if the 'recipient' is an array and handling this scenario accordingly in the UI.

The displayed 'recipient' now can handle single or multiple recipients (displayed as a list), specified with their address and percentage share. Additionally, error handling has been added to catch any exception caused by JSON parsing the 'recipient' data.

Additional styles have also been added to adjust the look of the recipient section, which includes flex-direction, alignment of items, text transformation, and color among others.

@reviewpad reviewpad bot added small Pull request is small waiting-for-review labels Aug 28, 2023
@roiLeo roiLeo requested a review from prury August 29, 2023 06:35
components/gallery/GalleryItemDescription.vue Outdated Show resolved Hide resolved
components/gallery/GalleryItemDescription.vue Outdated Show resolved Hide resolved
components/gallery/GalleryItemDescription.vue Outdated Show resolved Hide resolved
components/gallery/GalleryItemDescription.vue Outdated Show resolved Hide resolved
components/gallery/GalleryItemDescription.vue Outdated Show resolved Hide resolved
components/gallery/GalleryItemDescription.vue Outdated Show resolved Hide resolved
components/gallery/GalleryItemDescription.vue Outdated Show resolved Hide resolved
components/gallery/GalleryItemDescription.vue Outdated Show resolved Hide resolved
@yangwao
Copy link
Member

yangwao commented Aug 29, 2023

@niklasp please fix your PR with a PR template. Thanks

Converted to draft till then.

@yangwao yangwao marked this pull request as draft August 29, 2023 10:02
@niklasp
Copy link
Contributor Author

niklasp commented Aug 29, 2023

@yangwao used template
@roiLeo merged suggestions and fixed ol issue

@niklasp niklasp marked this pull request as ready for review August 29, 2023 22:43
@codeclimate
Copy link

codeclimate bot commented Aug 29, 2023

Code Climate has analyzed commit 71d70fb and detected 0 issues on this pull request.

View more on Code Climate.

@sonarcloud
Copy link

sonarcloud bot commented Aug 29, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@niklasp
Copy link
Contributor Author

niklasp commented Aug 29, 2023

@prury @preschian ready for review

@yangwao yangwao changed the title fix #6970 fix multi recipient royalties on Details Tab Aug 30, 2023
@yangwao
Copy link
Member

yangwao commented Aug 30, 2023

pay 50 usd
can go for next release

@yangwao
Copy link
Member

yangwao commented Aug 30, 2023

😍 Perfect, I’ve sent the payout
💵 $50 @ 4.58 USD/DOT ~ 10.917 $DOT
🧗 1soESeTVLfse9e2G8dRSMUyJ2SWad33qhtkjQTv9GMToRvP
🔗 0x86eaab8542e54a095fc96794c65c4cce3218f1e851dd0c2834efc56689b2f7bc

🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues

@yangwao yangwao added paid pull-request has been paid R-expected-for-the-next-release labels Aug 30, 2023
@prury
Copy link
Member

prury commented Aug 30, 2023

pay 50 usd can go for next release

cool, before that we have to add fields on the NFT creation page to add more than one royalty recipient and percentages

@prury prury added the S-works-for-me-✅ qa-guild has tested PR from end user perspective and functionality worked label Aug 30, 2023
@yangwao yangwao merged commit ead6673 into kodadot:main Aug 30, 2023
15 checks passed
@niklasp
Copy link
Contributor Author

niklasp commented Aug 30, 2023

thanks for the assistance

This was referenced Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
paid pull-request has been paid R-expected-for-the-next-release S-works-for-me-✅ qa-guild has tested PR from end user perspective and functionality worked small Pull request is small waiting-for-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

multi recipient royalties on Details Tab
4 participants