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

React hooks for main containers (dashboard, footer, header, route) #893

Closed
huss opened this issue Apr 19, 2023 · 2 comments
Closed

React hooks for main containers (dashboard, footer, header, route) #893

huss opened this issue Apr 19, 2023 · 2 comments
Milestone
2.0 release

Comments

@huss
Copy link
Member

huss commented Apr 19, 2023

Is your feature request related to a problem? Please describe.

Please see issue #726 for information.

Describe the solution you'd like

This transitions a number of components where the code located in files src/client/app/components/DashboardComponent.tsx, src/client/app/containers/DashboardContainer.ts, src/client/app/components/FooterComponent.tsx, src/client/app/containers/FooterContainer.ts, src/client/app/components/HeaderComponent.tsx, src/client/app/containers/HeaderContainer.ts, src/client/app/components/RouteComponent.tsx & src/client/app/containers/RouteContainer.ts need to be updated for React hooks. The look of each item will stay the same.

Additional context

It is hoped that other pages do not need to be changed as part of this but these are used across the OED React code base.
@huss huss added this to the 1.1 release milestone Apr 19, 2023
@huss huss added the reserved Expected to be assigned to developer by project so others should not work on without prior approval. label Apr 19, 2023
@huss huss mentioned this issue May 9, 2023
Closed
10 tasks
@spearec
Copy link
Contributor

spearec commented Jul 13, 2023

#958 converts HeaderComponent to use hooks

@huss huss removed the reserved Expected to be assigned to developer by project so others should not work on without prior approval. label Sep 26, 2023
@ChrisMart21 ChrisMart21 self-assigned this Sep 26, 2023
@ChrisMart21 ChrisMart21 mentioned this issue Jan 15, 2024
Merged
5 tasks
@ChrisMart21 ChrisMart21 removed their assignment Mar 5, 2024
@huss
Copy link
Member Author

huss commented Mar 6, 2024

Fixed by PR #1113 so closing.

@huss huss closed this as completed Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.0 release
Development

No branches or pull requests
3 participants
@huss @ChrisMart21 @spearec