hackforla / hackforla/311-data

ER: Updating Pin's react class components to functional components (Performance)

Open
#1,914 7 comments 0 reactions 0 assignees View on GitHub
Complexity: Small Emergent Request p-feature: Map Role: Frontend size: 0.50pt
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.