search

airpartners / aq-web-client

Web app for end-users.
https://airpartners-ade.web.app/
2 stars 0 forks source link

Major refactor. #8

Closed ljordan51 closed 4 years ago

ljordan51 commented 4 years ago

Main motivation was to refactor routing and state so back/forward actions would work in browser and on device (Android).

List of notable changes:

  1. Refactored routing to use url params for page (device or static: about us, q&a) and subpage (home/map).
  2. Refactored state in all relevant components to use passed down url params. a. App no longer manages any state. Just a container for parameterized routes. b. Added ContentContainer to manage state based on route params. State passed to NavDrawer and DevicePage as props. c. ContentContainer manages async fetching of data and maintains fetching state. d. Removed setPath handler that was previously passed from App down to child components. e. Added useEffect hooks to DeviceMap to re-center on props update.
    1. Updated locale translation with routes field. Updated Link components to use them.
    2. Added link to MarkerComponent InfoWindow. Added class to prevent underline.
    3. ContentContainer and DevicePage return h1 components with 404 message for bad routes. Should update this to something prettier eventually.