-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add feedback redesign #2487
Add feedback redesign #2487
Conversation
PR deployed in Google Cloud |
PR deployed in Google Cloud |
517a559
to
5727c81
Compare
5727c81
to
d74b660
Compare
86724f2
to
ffee519
Compare
ffee519
to
2155682
Compare
48701d2
to
8d23257
Compare
@@ -24,6 +25,10 @@ export type MetaData = { | |||
} | |||
} | |||
|
|||
type TinlakeTranchesKey = 'silver' | 'blocktowerThree' | 'blocktowerFour' |
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.
type TinlakeTranchesKey = 'silver' | 'blocktowerThree' | 'blocktowerFour' | |
type TinlakeTranchesKey = 'NS3' | 'BT3' | 'BT4' |
@@ -50,6 +57,33 @@ const StyledCard = styled(Card)` | |||
} | |||
` | |||
|
|||
const tinlakeTranches = { | |||
silver: { |
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.
silver: { | |
NS3: { |
shortDescription: ' Real estate bridge loans for fix and flip projects, maturing in 12-24 months.', | ||
InvestorType: 'Qualified Investors', | ||
}, | ||
blocktowerThree: { |
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.
blocktowerThree: { | |
BT3: { |
shortDescription: ' Investment-grade consumer ABS, auto ABS, and CLOs under 4 years.', | ||
InvestorType: 'Private', | ||
}, | ||
blocktowerFour: { |
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.
blocktowerFour: { | |
BT4: { |
{text} | ||
</Text> | ||
) | ||
const isTinlakePool = |
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.
const isTinlakePool = | |
const isTinlakePool = poolId.startsWith('0x') |
this should be enough since only tinlake pools have hex coded IDs
}) | ||
.reverse() | ||
const tinlakeObjKey = () => { | ||
if (name?.includes('Silver')) return 'silver' |
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.
if (name?.includes('Silver')) return 'silver' | |
if (name?.includes('New Silver 3')) return 'NS3' |
I think you need to be more specific here since we have two other New Silver pools (in the archive)
.reverse() | ||
const tinlakeObjKey = () => { | ||
if (name?.includes('Silver')) return 'silver' | ||
else if (name?.includes('BlockTower Series 3')) return 'blocktowerThree' |
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.
else if (name?.includes('BlockTower Series 3')) return 'blocktowerThree' | |
else if (name?.includes('BlockTower Series 3')) return 'BT3' |
const tinlakeObjKey = () => { | ||
if (name?.includes('Silver')) return 'silver' | ||
else if (name?.includes('BlockTower Series 3')) return 'blocktowerThree' | ||
else if (name?.includes('BlockTower Series 4')) return 'blocktowerFour' |
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.
else if (name?.includes('BlockTower Series 4')) return 'blocktowerFour' | |
else if (name?.includes('BlockTower Series 4')) return 'BT4' |
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.
Nice work!! Let's merge this and I'll address the comments in a follow up PR
#2471
#2484