Skip to content

Commit 375e5a0

Browse files
committed
Resolved merge conflicts
2 parents b7599c8 + f0fb4f1 commit 375e5a0

24 files changed

+3295
-531
lines changed

.github/ISSUE_TEMPLATE/blank-design-issue.md

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@
22
name: Blank Design Issue
33
about: Template for designers to create issues and prepare for engineering hand off.
44
title: ''
5-
labels: 'Milestone: Missing, role: UI/UX Design, Size: Missing'
5+
labels: 'Complexity: Missing, draft, Feature: Missing, Milestone: Missing, role: UI/UX
6+
Design, size: Missing'
67
assignees: ''
78

89
---
@@ -12,9 +13,13 @@ We need to do X for Y reason.
1213

1314
### Action Items
1415

15-
- [ ] Address the problem overview with possible design solution(s) in a comment
1616
- [ ] Complete Design Iterations section below
17+
- [ ] design requirement 1
18+
- [ ] design requirement 2
19+
- [ ] design requirement 99
1720
- [ ] Document user interaction in Figma
21+
- [ ] Update the Hand Off section of this ticket with the final iteration of this design
22+
1823

1924
---
2025

.github/ISSUE_TEMPLATE/blank-dev-issue.md

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,12 @@
22
name: Blank Dev Issue
33
about: Template for developers to create issues and prepare for engineering tickets.
44
title: ''
5-
labels: 'draft, Feature: Missing, Milestone: Missing, Role: Frontend, Size: Missing'
5+
labels: 'Complexity: Missing, draft, Feature: Missing, Milestone: Missing, Role: Frontend,
6+
size: Missing'
67
assignees: ''
78

89
---
910

10-
<!-- @format -->
11-
1211
### Overview
1312

1413
We need to do X for Y reason.
@@ -26,18 +25,22 @@ Explain here if the issue requires additional info
2625

2726
### Resources/Instructions
2827

29-
[Replace this text - Provide these resources here if exist. Provide screenshot only if the issue has visual
30-
changes on the site, do not post screenshots of codebase]
28+
- Figma Section:
29+
- Related Design Ticket:
3130

32-
- Figma design file link
33-
- Related previous ticket
3431
<details><summary>Screenshot before proposed changes</summary>
3532
<p>
3633
[insert screenshot here]
3734
</p>
3835
</details>
3936

40-
<details><summary>Screenshot after proposed changes</summary>
37+
<details><summary>Screenshot of Figma</summary>
38+
<p>
39+
[insert screenshot here]
40+
</p>
41+
</details>
42+
43+
<details><summary>Screenshot of new feature from localhost</summary>
4144
<p>
4245
[insert screenshot here]
4346
</p>

.github/ISSUE_TEMPLATE/blank-issue.md

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@
22
name: Blank Issue
33
about: Describe this issue's purpose here
44
title: ''
5-
labels: 'Milestone: Missing, Role: Missing, Size: Missing'
5+
labels: 'Complexity: Missing, draft, Feature: Missing, Milestone: Missing, Role: Missing,
6+
size: Missing'
67
assignees: ''
78

89
---
@@ -12,5 +13,9 @@ We need to do X for Y reason.
1213

1314
### Action Items
1415

16+
- [ ] Action Item 1
17+
- [ ] Action Item 2
18+
- [ ] Action Item 99
19+
1520
### Resources/Instructions
1621
REPLACE THIS TEXT -If there is a website which has documentation that helps with this issue provide the link(s) here.

.github/ISSUE_TEMPLATE/blank-research-plan.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@
22
name: Blank Research Plan
33
about: Template for UX researcher to create research plan.
44
title: ''
5-
labels: 'Feature: Research, Milestone: Missing, role: UI/UX Research, Size: Missing'
5+
labels: 'Complexity: Missing, Feature: Research, Milestone: Missing, role: UI/UX Research,
6+
size: Missing'
67
assignees: ''
78

89
---

.github/ISSUE_TEMPLATE/blank_epic.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@
22
name: New Blank Epic
33
about: Template for PMs to create Epics to track batches of features in a milestone
44
title: ''
5-
labels: 'draft, Epic, Feature: Missing, Milestone: Missing, Role: Product Management,
6-
size: epic'
5+
labels: 'Complexity: Small, draft, Epic, Feature: Missing, Milestone: Missing, Role:
6+
Missing, size: 0.25pt'
77
assignees: ''
88

99
---

.github/ISSUE_TEMPLATE/bug_report.md

Lines changed: 0 additions & 38 deletions
This file was deleted.

.github/ISSUE_TEMPLATE/emergent-request.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,13 @@ name: Emergent Request
33
about: When you discover something in your issue that is out of scope and it needs
44
a new issue or discussion.
55
title: 'ER: (title, required)'
6-
labels: 'Complexity: Missing, Discussion, Emergent Request, Feature: Missing, Question,
6+
labels: 'Complexity: Small, Discussion, Emergent Request, Feature: Missing, Question,
77
Role: Missing, size: 0.25pt'
88
assignees: ''
99

1010
---
1111

12-
### Emergent Requirement - Problem Description
12+
### Emergent Request - Problem Description
1313
- (short-answer, required)
1414

1515
### Relevant Issue(s)

.gitignore

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ docker/
3030
venv/
3131
.vscode
3232
.editorconfig
33+
.prettierignore
3334

3435
# socrata experimental files
3536
*-socrata*
@@ -40,4 +41,5 @@ venv/
4041
# archive
4142
/archive/aws/API\ Responses/*
4243

43-
.DS_Store
44+
.DS_Store
45+

README.md

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,12 +65,21 @@ Our mission is to create a user-friendly platform for anyone interested in explo
6565
- Run `npm start`
6666
- Visit http://localhost:5173
6767

68+
### Run tests
69+
70+
- Run `npm test`
71+
72+
See [contributing.md](contributing.md#testing) for info on writing tests.
73+
6874
### Information About Technologies
6975

7076
- Frontend
7177
- Mapbox
7278
- React
7379
- MUI
80+
- Vite
81+
- Vitest
82+
- React Testing Library
7483
- Backend
7584
- DuckDb
7685
- HuggingFace

components/Map/Map.jsx

Lines changed: 21 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,7 @@ const styles = (theme) => ({
8181
marginRight: 5,
8282
},
8383
'& .mapboxgl-popup': {
84+
maxWidth: '474px!important',
8485
'&-anchor-bottom .mapboxgl-popup-tip': {
8586
borderTopColor: theme.palette.primary.main,
8687
},
@@ -95,6 +96,11 @@ const styles = (theme) => ({
9596
},
9697
},
9798
},
99+
loadedModal: {
100+
'& .mapboxgl-popup-content': {
101+
borderRadius: 30,
102+
},
103+
},
98104
menuWrapper: {
99105
position: 'absolute',
100106
left: '20px',
@@ -415,7 +421,7 @@ class Map extends React.Component {
415421

416422
addPopup = (coordinates, requestId) => {
417423
this.setState({ selectedRequestId: requestId });
418-
this.popup = new mapboxgl.Popup()
424+
this.popup = new mapboxgl.Popup({closeButton: false, anchor: 'left'})
419425
.setLngLat(coordinates)
420426
.setDOMContent(this.requestDetail)
421427
.addTo(this.map);
@@ -430,7 +436,7 @@ class Map extends React.Component {
430436
};
431437

432438
reset = () => {
433-
const { dispatchUpdateNcId } = this.props;
439+
const { dispatchUpdateNcId, dispatchUpdateSelectedCouncils } = this.props;
434440

435441
this.zoomOut();
436442
this.addressLayer.clearMarker();
@@ -456,23 +462,18 @@ class Map extends React.Component {
456462
});
457463

458464
// Reset MinZoom to original value after deselecting NC
465+
this.resetBoundaries();
466+
dispatchUpdateSelectedCouncils([])
459467
this.map.setMinZoom(DEFAULT_MIN_ZOOM);
460468
};
461469

462470
resetBoundaries = () => {
463471
const {
464472
dispatchUpdateNcId,
465-
dispatchUpdateSelectedCouncils,
466-
dispatchUpdateUnselectedCouncils,
467-
councils,
468473
} = this.props;
469474

470475
// Reset the selected NcId back to null.
471476
dispatchUpdateNcId(null);
472-
473-
// Reset councilSelector.
474-
dispatchUpdateSelectedCouncils([]);
475-
dispatchUpdateUnselectedCouncils(councils);
476477
};
477478

478479
addressSearchIsEmpty = () => {
@@ -765,6 +766,13 @@ class Map extends React.Component {
765766
}
766767
}
767768
}
769+
setRequestDetailLoading = (isLoading) => {
770+
if (!isLoading && this.popup) {
771+
this.popup.addClassName(this.props.classes.loadedModal);
772+
}
773+
};
774+
775+
//// RENDER ////
768776

769777
render() {
770778
const {
@@ -815,7 +823,10 @@ class Map extends React.Component {
815823
boundaryStyle={mapStyle === 'dark' ? 'light' : 'dark'}
816824
/>
817825
<div ref={(el) => (this.requestDetail = el)}>
818-
<RequestDetail requestId={selectedRequestId} />
826+
<RequestDetail
827+
requestId={selectedRequestId}
828+
loadingCallback={this.setRequestDetailLoading}
829+
/>
819830
</div>
820831
{this.state.mapReady && requestTypes && (
821832
<>

0 commit comments

Comments
 (0)