Skip to content

Commit 35ab5ab

Browse files
committed
Remove conflicts and pushed_conflicts states
1 parent 655c923 commit 35ab5ab

File tree

2 files changed

+27
-16
lines changed

2 files changed

+27
-16
lines changed

Diff for: src/components/LayerSwitcher/index.tsx

-1
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,6 @@ function LayerSwitcher(props: Props) {
136136
label="Layer URL"
137137
value={layerUrl}
138138
onChange={setLayerUrl}
139-
hint="http://tile.stamen.com/watercolor/{z}/{x}/{y}.jpg"
140139
/>
141140
</ModalBody>
142141
<ModalFooter>

Diff for: src/views/Dashboard/index.tsx

+27-15
Original file line numberDiff line numberDiff line change
@@ -56,19 +56,33 @@ enum PosmStateEnum {
5656
'detecting_conflicts',
5757
'creating_geojsons',
5858

59-
'conflicts',
60-
'resolved',
59+
'resolving_conflicts',
6160
'pushing_conflicts',
62-
'pushed_upstream',
6361
}
6462

6563
const AOI_POLL_TIME = 3000;
6664

6765
const isNotStarted = (state: PosmStateEnum) => state <= PosmStateEnum.not_triggered;
68-
const isConflicted = (state: PosmStateEnum) => state === PosmStateEnum.conflicts;
69-
const isResolved = (state: PosmStateEnum) => state === PosmStateEnum.resolved;
70-
const isPushing = (state: PosmStateEnum) => state === PosmStateEnum.pushing_conflicts;
71-
const isPushed = (state: PosmStateEnum) => state === PosmStateEnum.pushed_upstream;
66+
67+
const isConflicted = (
68+
state: PosmStateEnum,
69+
isCurrentStateComplete?: boolean,
70+
) => state === PosmStateEnum.resolving_conflicts && !isCurrentStateComplete;
71+
72+
const isResolved = (
73+
state: PosmStateEnum,
74+
isCurrentStateComplete?: boolean,
75+
) => state === PosmStateEnum.resolving_conflicts && isCurrentStateComplete;
76+
77+
const isPushing = (
78+
state: PosmStateEnum,
79+
isCurrentStateComplete?: boolean,
80+
) => state === PosmStateEnum.pushing_conflicts && !isCurrentStateComplete;
81+
82+
const isPushed = (
83+
state: PosmStateEnum,
84+
isCurrentStateComplete?: boolean,
85+
) => state === PosmStateEnum.pushing_conflicts && isCurrentStateComplete;
7286

7387
interface PosmState {
7488
id: PosmStateEnum;
@@ -337,10 +351,8 @@ class Dashboard extends React.PureComponent<Props, State> {
337351
{ id: PosmStateEnum.extracting_local_aoi, name: 'Extracting Local AOI' },
338352
{ id: PosmStateEnum.detecting_conflicts, name: 'Identifying conflicts' },
339353
{ id: PosmStateEnum.creating_geojsons, name: 'Creating GeoJSONs' },
340-
{ id: PosmStateEnum.conflicts, name: 'Resolving conflicts', inanimate: true },
341-
{ id: PosmStateEnum.resolved, name: 'Resolved conflicts', hidden: true },
354+
{ id: PosmStateEnum.resolving_conflicts, name: 'Resolving conflicts' },
342355
{ id: PosmStateEnum.pushing_conflicts, name: 'Pushing resolved data to OSM' },
343-
{ id: PosmStateEnum.pushed_upstream, name: 'Resolved data pushed to OSM', hidden: true },
344356
],
345357
alreadyLoaded: false,
346358
};
@@ -518,10 +530,10 @@ class Dashboard extends React.PureComponent<Props, State> {
518530
} = this.state;
519531

520532
const notStartedStep = isNotStarted(posmStatus.state);
521-
const conflictedStep = isConflicted(posmStatus.state);
522-
const resolvedStep = isResolved(posmStatus.state);
523-
const pushingStep = isPushing(posmStatus.state);
524-
const pushedStep = isPushed(posmStatus.state);
533+
const conflictedStep = isConflicted(posmStatus.state, posmStatus.isCurrentStateComplete);
534+
const resolvedStep = isResolved(posmStatus.state, posmStatus.isCurrentStateComplete);
535+
const pushingStep = isPushing(posmStatus.state, posmStatus.isCurrentStateComplete);
536+
const pushedStep = isPushed(posmStatus.state, posmStatus.isCurrentStateComplete);
525537

526538
let conflictProgress = 0;
527539
if (totalResolvedElements && totalConflictingElements) {
@@ -750,7 +762,7 @@ class Dashboard extends React.PureComponent<Props, State> {
750762
{((conflictedStep || resolvedStep) && !posmStatus.hasErrored) && (
751763
<div className={styles.checkboxesContainer}>
752764
<Checkbox
753-
label="Show Conflicts in Map"
765+
label="Show conflicted elements"
754766
value={conflictsVisibility}
755767
onChange={this.handleShowConflictsButtonClick}
756768
/>

0 commit comments

Comments
 (0)