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.

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

Refactor #74

Closed
wants to merge 12 commits into from
Closed

Refactor #74

wants to merge 12 commits into from

Conversation

palagdan
Copy link
Collaborator

@palagdan palagdan commented Nov 1, 2024

#62

Replace App component with Router for routing
Update react-router-dom to v6
Refactor navbar, footer into separate folders and replace styles with the mui library
Refactor and move Home component to HomePage
Replace semantic ui with mui
Refactor Scripts class component to functional ScriptsPage component.
Move to folder pages
Replace NoMatch Component with NotFoundPage
Refactor Executions class component to functional ExecutionsPage component.
Move to folder pages
Replace Bootstrap with Mui
Replace Fortawesome with Mui icons
Move vocabulary constants from Rest file to the separate vocabulary file.
Install Axios for API calls.
Refactor the Rest module.
Define API URL constants.
Implement fetchers for Axios calls.
@blcham
Copy link
Contributor

blcham commented Nov 12, 2024

@palagdan what is this PR ?

@palagdan
Copy link
Collaborator Author

@blcham
This was the initial PR for the refactoring. I decided to split it into separate issues because it was difficult to work on.
I close this PR.

@palagdan palagdan closed this Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants