-
Notifications
You must be signed in to change notification settings - Fork 22
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
adding sub-tabs for listings in progress #174
Conversation
…to can challenge be resolved removing logs running prettier
f25d3b5
to
e053451
Compare
Pull Request Test Coverage Report for Build 3290
💛 - Coveralls |
@@ -63,5 +63,5 @@ export function canResolveChallenge(challengeData: ChallengeData): boolean { | |||
* @param challengeData this ChallengeData to check | |||
*/ | |||
export function doesChallengeHaveAppeal(challengeData: ChallengeData): boolean { | |||
return !challengeData.appeal || is0x0Address(challengeData!.appeal!.requester.toString()); | |||
return challengeData.appeal! && !is0x0Address(challengeData!.appeal!.requester.toString()); |
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.
Durrrr. Thanks for fixing this.
|
||
class ListingsInProgress extends React.Component<ListingProps> { | ||
public render(): JSX.Element { | ||
const applications = this.props.applications; |
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.
should these be represented in the reducers? Like instead of merging them here, get rid of the smaller categories and replace them with meaningful sets?
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.
I'd rather keep the small categories at least for now. Don't think we're even sure what exactly the bigger categories would be, this makes it a bit easier to change things around?
also: