-
Notifications
You must be signed in to change notification settings - Fork 15
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
feat: add teams filtration for streaming payments page #4057
feat: add teams filtration for streaming payments page #4057
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome work on this one @adam-strzelec 🥇
Previously created 2 active streaming payments and one that has not yet started in Rocinate
View for All teams
View for General
View for Rocinate
View for Andromeda
where there are no streams
I left a small refactoring comment so we keep it consistent as much as possible with the other places where we use the team filtering 💯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we please use here ContentWithTeamFilter
instead of TeamFilter
?
So we could refactor this as
<ContentWithTeamFilter>
<StatsCards
streamingPerMonth={totalLastMonthStreaming}
totalActiveStreams={activeStreamingPayments}
totalStreamed={totalStreamed}
unclaimedFounds={totalFunds.totalAvailable}
prefix={currencySymbolMap[currency]}
suffix={currency}
/>
</ContentWithTeamFilter>
@adam-strzelec There is already a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @adam-strzelec thank you for your super-fast changes! 💯
However I had in mind re-using the existing ContentWithTeamFilter
component as there is no value into creating a new one 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we please use the existing component ContentWithTeamFilter
from src/components/v5/frame/ContentWithTeamFilter/ContentWithTeamFilter.tsx
?
And if we really need the distance between the filter and content to be of 8
, we could refactor the component as in
import React, { type FC, type PropsWithChildren } from 'react';
import TeamFilter from '~v5/shared/TeamFilter/TeamFilter.tsx';
const displayName = 'v5.frame.ContentWithTeamFilter';
interface ContentWithTeamFilterProps extends PropsWithChildren {
className?: string;
}
const ContentWithTeamFilter: FC<ContentWithTeamFilterProps> = ({
className = 'pb-6',
children,
}) => {
return (
<div className="flex flex-col">
<div className={className}>
<TeamFilter />
</div>
{children}
</div>
);
};
ContentWithTeamFilter.displayName = displayName;
export default ContentWithTeamFilter;
/> | ||
</div> | ||
</> | ||
<ContentWithTeamFilter> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And then pass here the className='pb-8'
@mmioana @iamsamgibbs Right, I didn't notice that this component already exists 🤦♂️ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your continuous effort @adam-strzelec 🙌 everything works as expected
Screen.Recording.2025-01-13.at.16.07.04.mov
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ab28bf5
into
feat/streaming-payments-ui
Description
Add filtration for streaming payments page
Testing
Open
Streaming payments
pageDiffs
New stuff ✨
Resolves #4021