LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

More stripe integration #6649

Closed
mayeaux wants to merge 17 commits from more-stripe-integration into master
pull from: more-stripe-integration
merge into: LBRYCommunity:master
Conversation 1 Commits 17 Files changed 7 +38 -2
1 changed files with 38 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit df62e9c0db - Show all commits

40
ui/component/livestreamComments/view.jsx
View file

@ -24,6 +24,7 @@ type Props = {
superChats: Array<Comment>, superChats: Array<Comment>,
superChatsReversed: Array, superChatsReversed: Array,
jessopb commented 2021-07-29 23:16:09 +02:00 (Migrated from github.com)
Review
Copy link

This does'nt seem to be passed in. Does it belong in props types?

This does'nt seem to be passed in. Does it belong in props types?
superChatsTotalAmount: number, superChatsTotalAmount: number,
superChatsFiatAmount: number,
myChannels: ?Array<ChannelClaim>, myChannels: ?Array<ChannelClaim>,
}; };
@ -48,7 +49,42 @@ export default function LivestreamComments(props: Props) {
superChats, // superchats organized by tip amount superChats, // superchats organized by tip amount
} = props; } = props;
let { superChatsReversed } = props; let { superChatsReversed, superChatsFiatAmount } = props;
superChatsFiatAmount = 0;
if(superChats){
console.log(superChats);
let fiatAmount = 0;
for(const superChat of superChats){
if(superChat.is_fiat){
jessopb commented 2021-07-29 23:17:20 +02:00 (Migrated from github.com)
Review
Copy link

One thing you can do is get
{ superChats: superchatsSortedByAmount, comments: superchatsSortedByDate } = props; to use a new name after destructuring.

One thing you can do is get { superChats: superchatsSortedByAmount, comments: superchatsSortedByDate } = props; to use a new name after destructuring.
jessopb commented 2021-07-29 23:19:59 +02:00 (Migrated from github.com)
Review
Copy link

Perhaps whatever is connect()ed in in the index.js could just have a better name.

Perhaps whatever is connect()ed in in the index.js could just have a better name.
fiatAmount = fiatAmount + superChat.support_amount;
}
}
superChatsFiatAmount = fiatAmount;
}
// TODO: why doesn't this work?
React.useEffect(() => {
if(superChats){
console.log(superChats);
// let fiatAmount = 0;
// for(const superChat of superChats){
// if(superChat.is_fiat){
// fiatAmount = fiatAmount + superChat.support_amount;
// }
// }
//
// console.log(fiatAmount);
//
// superChatsFiatAmount = fiatAmount.toString();
}
}, [superChats]);
if (superChats) { if (superChats) {
const clonedSuperchats = JSON.parse(JSON.stringify(superChats)); const clonedSuperchats = JSON.parse(JSON.stringify(superChats));
@ -172,7 +208,7 @@ export default function LivestreamComments(props: Props) {
label={ label={
<> <>
<CreditAmount amount={superChatsTotalAmount} size={8} /> / <CreditAmount amount={superChatsTotalAmount} size={8} /> /
<CreditAmount amount={superChatsTotalAmount} size={8} isFiat={true} /> {' '}{__('Tipped')} <CreditAmount amount={superChatsFiatAmount} size={8} isFiat={true} /> {' '}{__('Tipped')}
</> </>
} }
onClick={function() { onClick={function() {