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

Token launch CTA elements #9317

Merged
merged 5 commits into from
Sep 25, 2024
Merged

Conversation

mzparacha
Copy link
Contributor

Link to Issue

Closes: #8704

Description of Changes

  • Add token launch button to content creation sidebar, popover, and mobile drawer.
  • Add token launch card on dashboard page for non mobile view only

"How We Fixed It"

N/A

Test Plan

  1. Enable FLAG_TOKENIZED_COMMUNITY
  2. Visit the dashboard page, and verify you see the token launch card, click on the launch token button and verify you get redirected to /createTokenCommunity
  3. Open the content creation sidebar from the + icon in app sidebar, and verify you see the token launch button
  4. Visit any community, open the content creation popover and verify you see the token launch button
  5. Resize to mobile screen, open the content creation drawer and verify you see the token launch button

Deployment Plan

N/A

Other Considerations

N/A

@mzparacha mzparacha requested review from masvelio, Israellund and burtonator and removed request for masvelio and Israellund September 23, 2024 19:33
@github-actions github-actions bot marked this pull request as draft September 23, 2024 19:33
@mzparacha mzparacha marked this pull request as ready for review September 23, 2024 19:37
Copy link
Contributor

@masvelio masvelio left a comment

Choose a reason for hiding this comment

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

  1. cant see the card on mobile

image

@mzparacha
Copy link
Contributor Author

Thanks for the review @masvelio, I fixed the token launch card not showing on the mobile screens.

Copy link
Collaborator

@Israellund Israellund left a comment

Choose a reason for hiding this comment

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

LGTM!

@mzparacha mzparacha merged commit 62533b7 into master Sep 25, 2024
10 checks passed
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

Successfully merging this pull request may close these issues.

Token Launch CTA elements
3 participants