Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Release update #458

Merged
merged 51 commits into from
Mar 25, 2020
Merged

Release update #458

merged 51 commits into from
Mar 25, 2020

Conversation

sellnat77
Copy link
Member

This includes front end features
Backend DB connection fixes
Backend route fixes

rgao and others added 30 commits February 12, 2020 17:58
@rgao
framework for precache file
fd7b45b
@rgao
Merge remote-tracking branch 'upstream/dev' into dev
5573b9b
@rgao
implemented method to query recently added entries for precaching req…
1cc35a8 
…uest count bar graph; reverted connection string to postgres
@rgao
Merge remote-tracking branch 'upstream/dev' into dev
2190233
@rgao
pre-compiled recent datasets of created dates for each request type /…
25102d8 
… council
@rgao
Merge remote-tracking branch 'upstream/dev' into dev
417c67a
@rgao
updated query string for precaching
11476df
@rgao
updated from upstream
2e5de41
@rgao
updated frequency module, fixed typo on sqlingest
c0fb61d
@rgao
fixed merge conflict in app.py
37c6818
@rgao
fixed merge issues in app.py and fixed [most of] the issues in json r…
881c240 
…esponse
@rgao
fixed all linting errors, specified POST method for frequency module …
cbde943 
…path
@adamkendis
Implemented fetchPinInfo worker and getPinData saga.
7bee5ba
@jmensch1
fixed console errors generated by proptypes
848c1be
@jmensch1
added show more/show less toggle to Criteria
a64f192
@jmensch1
export for charts
7b3024e
@jmensch1
export for map
a3dc5ee
@sellnat77
Removed terraform checks
0b3c962
@sellnat77
Merge branch 'dev' into dev
aa5218d
@sellnat77
Merge pull request #432 from rgao/dev
c88655d 
Updated Frequency Module
@sellnat77
Merge branch 'dev' into disableTerraform
eb85ba1
@jmensch1
comments and cleanup
39b2340
@jmensch1
Merge pull request #454 from hackforla/disableTerraform
b5d4aca 
Removed terraform checks
@jmensch1
Merge branch 'dev' into 411-FRONT-ExportFunctionality
58da97b
@jmensch1
Merge branch '411-FRONT-ExportFunctionality' of https://github.com/ha…
38b0a0b 
…ckforla/311-data into 411-FRONT-ExportFunctionality
@jmensch1
time to close and frequency fixes
e8f6409
@jmensch1
closing DB sessions after queries
49ba27c
@jmensch1
updated pytest for time to close with new name
259fb21
@sellnat77
Merge pull request #455 from hackforla/BACK-TimeToCloseAndFrequencyFixes
916e428 
Back time to close and frequency fixes
@jmensch1
Merge branch 'dev' into 411-FRONT-ExportFunctionality
87b6faf
adamkendis and others added 19 commits March 23, 2020 21:02
@adamkendis
Added fullType property to a few shortened REQUEST_TYPES constants.
4548179
@adamkendis
Custom PinPopup component.
c3d4787
@adamkendis
Pinpopup stylesheet. Added pinpopup to main styles imports.
60780d5
@adamkendis
Added custom marker popup to PinMap. Mapped getPinInfoRequest to props.
d809af8
@adamkendis
PinInfo related action creators.
45753ff
@adamkendis
Removed comments. Updated pin saga.
74b69d8
@jmensch1
live data for Frequency and Total Requests
67ac686
@adamkendis
Removed leaflet-markercluster cdn stylesheet from index. Imported sty…
8e47a17 
…les via sass.
@adamkendis
Added loader to PinPopup.
0364f3d
@adamkendis
Removed UI blocking loader from GET_PIN_INFO_REQUEST reducer.
e2f5971
@adamkendis
Added CustomMarker to PinMap. Tweaked custom marker textShadow. Reduc…
8923612 
…ed MarkerClusterGroup maxClusterRadius to 40.
@adamkendis
CustomMarker component.
1279846
@adamkendis
Removed REQUEST_TYPES constants lookup from PinPopup and moved to Pin…
127190b 
…Map. Color and abbrev now passed to PinPopup as props.
@adamkendis
Minor change to pinpopup styles.
7df2477
@jmensch1
using Button component instead of <button>
af7aa25
@adamkendis
Tweaked reducer for GET_PIN_INFO_REQUEST action.
aa38667
@jmensch1
Merge pull request #451 from hackforla/411-FRONT-ExportFunctionality
8cfcd32 
411 front export functionality
@adamkendis
Merge branch 'dev' into 332-FRONT-mapPinInfo
967d4ad
@brodly
Merge pull request #457 from adamkendis/332-FRONT-mapPinInfo
b02fb17 
332, 412 Map - custom markers, pin popup data fetching, clustering tweak
@sellnat77 sellnat77 requested review from jmensch1 and brodly March 24, 2020 20:07
@brodly
Copy link
Member

brodly commented Mar 24, 2020

Need to do some more testing with our current build before we publish, but found an issue with a the backend call /requestfrequency not working with POST requests. Error is:Failed to load resource: the server responded with a status of 405 (Method Not Allowed)

@sellnat77
Copy link
Member Author

Was that tested against latest dev? Looks like post is accepted https://github.com/hackforla/311-data/blob/dev/server/src/app.py#L71

@jmensch1
Copy link
Contributor

Yeah it's taking posts now, I don't think the previous version did

@sellnat77 sellnat77 merged commit d191dcd into master Mar 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brodly brodly brodly approved these changes

@jmensch1 jmensch1 Awaiting requested review from jmensch1

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@sellnat77 @brodly @jmensch1 @rgao @adamkendis