hackforla / hackforla/311-data
ER: Updating Pin's react class components to functional components (Performance)
- Dominant language
- JavaScript
- Stars
- 73
- Forks
- 74
- PR merge metrics
- No merged PRs in 30d
Description
### Emergent Requirement - Updating the map Pins for better performance
- We want to update the map Pins code from class components to functional components to see if this will make the application more performant.
### Relevant Issue(s)
- https://github.com/hackforla/311-data/issues/1865
### Date discovered
- 2/3/25
### Did this require a temporary workaround? If yes, what was it?
- N
### Who was involved
- @efrenmarin45 @ryanfchase @traycn
### What happens if this is not addressed
- The app continues to work, but will still apply the Pins logic in class component.
### Resources
- Redux Docs: [(Using Saga Helpers)](https://redux-saga.js.org/docs/basics/UsingSagaHelpers/)
- Mozilla Docs: [function*](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/function*)
- React Docs: [Migrating from a PureComponent class component to a function](https://react.dev/reference/react/PureComponent#migrating-from-a-purecomponent-class-component-to-a-function)
### Recommended Action Items
- [x] Share issue link with team Lead
- [ ] Add to upcoming agenda
### Potential Solutions
- Update the following files: `components/maps/index.js (L500)` and `redux/reducers/data.js`
Contributor guide
Assessment
This issue has not been assessed yet.