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

Follow up for success drop notif #6252

Closed
exezbcz opened this issue Jun 16, 2023 · 5 comments · Fixed by #6256
Closed

Follow up for success drop notif #6252

exezbcz opened this issue Jun 16, 2023 · 5 comments · Fixed by #6256
Assignees
Labels
A-notification p3 non-core, affecting less than 40% unlockable

Comments

@exezbcz
Copy link
Member

exezbcz commented Jun 16, 2023

Let's solve few things and add a Twitter redirect to the success notification
image

Visual changes

  • text align center
  • make the container bigger, like in the Figma - updated there

Functional

  • let's make the counter dynamic so that it will change every second. 29,28,....

  • adding a new sentence with Twitter redirect text link.

  • it will redirect you to Twitter with a tweet preset. Maybe @vikiival can do some magic and display an item/image that was already minted

  • copy:

      			Just minted an exclusive NFT with unlockable items on @Kodadot! 🎉 So excited to add this unique collectible to my collection. Don't miss your chance!
    
  • I guess I need some help here from @JustLuuuu 🥺

@exezbcz exezbcz added p3 non-core, affecting less than 40% A-notification unlockable labels Jun 16, 2023
@Jarsen136
Copy link
Contributor

Functional

  • let's make the counter dynamic so that it will change every second. 29,28,....

The minting time is estimated, which means it's not equal to 30 seconds. So should it be dynamically changed?

  • adding a new sentence with Twitter redirect text link.
  • it will redirect you to Twitter with a tweet preset. Maybe @vikiival can do some magic and display an item/image that was already minted

How about adding the URL of nft details page to twitter content ? https://kodadot.xyz/stmn/gallery/8-86. Though I found that there is no preview card on Twitter for this url.

@kodabot
Copy link
Collaborator

kodabot commented Jun 16, 2023

ASSIGNED - @Jarsen136 🔒 LOCKED -> Sunday, June 18th 2023, 24:21:01 UTC -> 36 hours

@exezbcz
Copy link
Member Author

exezbcz commented Jun 16, 2023

@Jarsen136 yes

changing the "in just 30 seconds" to " by now" should work

How about adding the URL of nft details page to twitter content ? https://kodadot.xyz/stmn/gallery/8-86. Though I found that there is no preview card on Twitter for this url.

the thing is that we don't have it ready afaik, but would be the best, with thumbnail as well

@exezbcz
Copy link
Member Author

exezbcz commented Jun 16, 2023

@Jarsen136 Also, I tried minting from nova wallet on mobile, clicked mint, and waited like 10 seconds while having the notif open. Then the notif disappeared and only toast was displayed "minted". Could you please check this type of behavior as well 👀

thanks a lot!

@Jarsen136
Copy link
Contributor

How about adding the URL of nft details page to twitter content ? https://kodadot.xyz/stmn/gallery/8-86. Though I found that there is no preview card on Twitter for this url.

the thing is that we don't have it ready afaik, but would be the best, with thumbnail as well

We don't have image preview for stmn items on twitter. I guess it will be solve in issue kodadot/workers#104 in the future. I could not add the thumbnail auto to the twitter content because it's not supported on the twitter side. User have to upload image manually.

image

ref: https://developer.twitter.com/en/docs/twitter-for-websites/tweet-button/guides/web-intent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-notification p3 non-core, affecting less than 40% unlockable
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants