Skip to content

Commit 6d0b3cc

Browse files
committed
mission planning: show error message (e.g. if not logged in) and recalculate assignments per person #33
1 parent 828d2e8 commit 6d0b3cc

File tree

1 file changed

+2
-11
lines changed

1 file changed

+2
-11
lines changed

client/components/MissionPlanning.js

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ export default React.createClass({
1717
missions: null,
1818
volunteers: null,
1919
error: null,
20-
message: null,
2120
};
2221
},
2322

@@ -38,7 +37,7 @@ export default React.createClass({
3837
const mappedVolunteers = _.keyBy(volunteers, 'id');
3938
this.setState({ missions, volunteers: mappedVolunteers }, this.setAssignments);
4039
})
41-
.catch(console.error);
40+
.catch(({ error }) => this.setState({ error }));
4241
},
4342

4443
setAssignments() {
@@ -48,16 +47,11 @@ export default React.createClass({
4847
this.setState({ assignments });
4948
},
5049

51-
setMessage(message) {
52-
this.setState({ message });
53-
_.delay(() => this.setState({ message: null }), 3000);
54-
},
55-
5650
updateMission(newMission) {
5751
const missions = this.state.missions;
5852
const index = _.findIndex(missions, mission => mission.id === newMission.id);
5953
missions[index] = newMission;
60-
this.setState({ missions });
54+
this.setState({ missions }, this.setAssignments);
6155
},
6256

6357
render() {
@@ -80,9 +74,6 @@ export default React.createClass({
8074
<img src="/images/logo.svg" height="32" alt="cadus crewing" style={{ marginTop: -10 }} />
8175
</Card>
8276
</header>
83-
{this.state.message &&
84-
<Alert type="success">{this.state.message}</Alert>
85-
}
8677
{_.map(this.state.missions, mission =>
8778
<Mission
8879
isEditable

0 commit comments

Comments
 (0)