Skip to content

Commit e078aa9

Browse files
Merge pull request #159 from yoution/issue-76
fix: issue #76
2 parents f890395 + 915ff0b commit e078aa9

File tree

2 files changed

+31
-31
lines changed

2 files changed

+31
-31
lines changed

src/containers/Challenges/Listing/index.jsx

Lines changed: 30 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ import PT from "prop-types";
33
import _ from "lodash";
44
import moment from "moment";
55
import Panel from "../../../components/Panel";
6+
import ChallengeError from "../Listing/errors/ChallengeError";
67
import Pagination from "../../../components/Pagination";
78
import ChallengeItem from "./ChallengeItem";
89
import TextInput from "../../../components/TextInput";
@@ -101,38 +102,39 @@ const Listing = ({
101102
</div>
102103
</div>
103104
</Panel.Header>
104-
<Panel.Body>
105-
{challenges.map((challenge, index) => (
106-
<div key={challenge.id} styleName={index % 2 === 0 ? "even" : "odd"}>
107-
<ChallengeItem
108-
challenge={challenge}
109-
onClickTag={(tag) => {
110-
const filterChange = { tags: [tag] };
111-
updateFilter(filterChange);
112-
}}
113-
onClickTrack={(track) => {
114-
const filterChange = { tracks: [track] };
105+
{challenges.length ?
106+
<Panel.Body>
107+
{challenges.map((challenge, index) => (
108+
<div key={challenge.id} styleName={index % 2 === 0 ? "even" : "odd"}>
109+
<ChallengeItem
110+
challenge={challenge}
111+
onClickTag={(tag) => {
112+
const filterChange = { tags: [tag] };
113+
updateFilter(filterChange);
114+
}}
115+
onClickTrack={(track) => {
116+
const filterChange = { tracks: [track] };
117+
updateFilter(filterChange);
118+
}}
119+
isLoggedIn={isLoggedIn}
120+
/>
121+
</div>
122+
))}
123+
<div styleName="pagination">
124+
<Pagination
125+
length={total}
126+
pageSize={perPage}
127+
pageIndex={utils.pagination.pageToPageIndex(page)}
128+
onChange={(event) => {
129+
const filterChange = {
130+
page: utils.pagination.pageIndexToPage(event.pageIndex),
131+
perPage: event.pageSize,
132+
};
115133
updateFilter(filterChange);
116134
}}
117-
isLoggedIn={isLoggedIn}
118135
/>
119136
</div>
120-
))}
121-
<div styleName="pagination">
122-
<Pagination
123-
length={total}
124-
pageSize={perPage}
125-
pageIndex={utils.pagination.pageToPageIndex(page)}
126-
onChange={(event) => {
127-
const filterChange = {
128-
page: utils.pagination.pageIndexToPage(event.pageIndex),
129-
perPage: event.pageSize,
130-
};
131-
updateFilter(filterChange);
132-
}}
133-
/>
134-
</div>
135-
</Panel.Body>
137+
</Panel.Body> : <ChallengeError /> }
136138
</Panel>
137139
);
138140
};

src/containers/Challenges/index.jsx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ import PT from "prop-types";
33
import { connect } from "react-redux";
44
import Listing from "./Listing";
55
import actions from "../../actions";
6-
import ChallengeError from "./Listing/errors/ChallengeError";
76
// import ChallengeRecommendedError from "./Listing/errors/ChallengeRecommendedError";
87
import * as constants from "../../constants";
98
import IconListView from "../../assets/icons/list-view.svg";
@@ -70,8 +69,7 @@ const Challenges = ({
7069
</button>
7170
</span>
7271
</h1>
73-
{challenges.length === 0 && initialized && <ChallengeError />}
74-
{challenges.length > 0 && (
72+
{initialized && (
7573
<>
7674
{/*noRecommendedChallenges && <ChallengeRecommendedError />*/}
7775
<Listing

0 commit comments

Comments
 (0)