Skip to content
This repository was archived by the owner on Oct 4, 2023. It is now read-only.

Add tipping audio to artists#1255

Merged
sddioulde merged 22 commits into
mainfrom
sd-tip-audio
May 4, 2022
Merged

Add tipping audio to artists#1255
sddioulde merged 22 commits into
mainfrom
sd-tip-audio

Conversation

@sddioulde

@sddioulde sddioulde commented Apr 27, 2022

Copy link
Copy Markdown
Contributor

Description

Add button and functionality to tip audio to users.

Dragons

N/A

How Has This Been Tested?

Locally against staging.

Went through entire flow, tipped audio to different users, check solscan.io to make sure transactions did go through and that balances were updated.

localhost_3001_saliou_2
localhost_3001_saliou_2 (1)
localhost_3001_saliou_2 (2)
localhost_3001_saliou_2 (3)
localhost_3001_saliou_2 (4)

How will this change be monitored?

Add metrics for tipping request/success/failure/twitter-share

@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/sd-tip-audio

@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/sd-tip-audio

1 similar comment
@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/sd-tip-audio

@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/sd-tip-audio

@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/sd-tip-audio

@sddioulde sddioulde requested review from jowlee and piazzatron April 29, 2022 16:40
@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/sd-tip-audio

@jowlee jowlee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Only looked at the store tipping saga - LGTM

@piazzatron piazzatron left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is epic!! Just a few little things I noticed when testing:

  • Tip button doesn't have a hover state
  • In the modal transition container, the back transition is super quick. Forward could use a bit more easing as well
  • At least in firefox, the 'available to send' text and badge aren't vertically aligned

Review comment are mostly nits and stylistic things. Looks great.

Comment thread packages/web/src/common/models/Tipping.ts
Comment thread packages/web/src/common/store/tipping/types.ts Outdated
Comment thread packages/web/src/common/store/tipping/slice.ts Outdated
Comment thread packages/web/src/components/tipping/tip-audio/ConfirmSendTip.tsx Outdated
Comment thread packages/web/src/components/tipping/tip-audio/SendTip.tsx
Comment thread packages/web/src/components/tipping/tip-audio/TipAudioModal.tsx Outdated
Comment thread packages/web/src/store/tipping/sagas.ts
Comment thread packages/web/src/store/tipping/sagas.ts
@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/sd-tip-audio

@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/sd-tip-audio

@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/sd-tip-audio

@audius-infra

Copy link
Copy Markdown
Collaborator

Preview this change https://demo.audius.co/sd-tip-audio

@sddioulde sddioulde merged commit 504a5c4 into main May 4, 2022
@sddioulde sddioulde deleted the sd-tip-audio branch May 4, 2022 23:17
@sliptype sliptype mentioned this pull request May 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants