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

Refactor after migration to Vite #62

Open
5 of 14 tasks
blcham opened this issue Aug 30, 2024 · 3 comments
Open
5 of 14 tasks

Refactor after migration to Vite #62

blcham opened this issue Aug 30, 2024 · 3 comments
Assignees
@palagdan
Labels
refactoring

Comments

@blcham
Copy link
Contributor

blcham commented Aug 30, 2024 •
edited by palagdan
Loading

To Do After Migration to Vite -- ordered by priority

  1. Resolve comments from PR -- Migration to Vite #61

  2. Fix websocket to support script changes detection #90

  3. Fix Script Execution Issues #96

  4. Refactor App component #78

  5. Refactor ScriptPage component #89

  6. Use axios library for API requests #86

  7. Fix start and finish date.  #95

  8. Refactor main.css  #94

  9. Refactor constants from Rest.tsx and Dagre.tsx to separate constants file #77

  10. Remove unused components and imports #81

  11. Refactor page components into a pages folder and implement user-friendly NotFoundPage #84

  12. Refactor to typescript #97

  13. Refactor Class Components to Functional Components #98

  14. Replace react-treebeard with a modern library #99
@blcham
Copy link
Contributor Author

blcham commented Aug 30, 2024

Also make sure that s-forms.css is imported and not duplicated in the project.

@palagdan
Copy link
Collaborator

palagdan commented Sep 3, 2024

blocked by #51

@LaChope LaChope self-assigned this Oct 17, 2024
palagdan added a commit that referenced this issue Nov 1, 2024
@palagdan
[#62] Refactor Router
19ce4f1 
Replace App component with Router for routing
Update react-router-dom to v6
palagdan added a commit that referenced this issue Nov 1, 2024
@palagdan
[#62] Refactor Layout
8335974 
Refactor navbar, footer into separate folders and replace styles with the mui library
palagdan added a commit that referenced this issue Nov 1, 2024
@palagdan
[#62] Refactor HomePage
82c657f 
Refactor and move Home component to HomePage
Replace semantic ui with mui
palagdan added a commit that referenced this issue Nov 1, 2024
@palagdan
[#62] Remove Profile Component
1291b82
palagdan added a commit that referenced this issue Nov 1, 2024
@palagdan
[#62] Refactor ScriptsPage
17f0b9f 
Refactor Scripts class component to functional ScriptsPage component.
Move to folder pages
palagdan added a commit that referenced this issue Nov 1, 2024
@palagdan
[#62] Implement NotFoundPage
5afecee 
Replace NoMatch Component with NotFoundPage
palagdan added a commit that referenced this issue Nov 1, 2024
@palagdan
[#62] Remove unused Features component
38172f2
palagdan added a commit that referenced this issue Nov 1, 2024
@palagdan
[#62] Refactor ExecutionsPage
1b7bedf 
Refactor Executions class component to functional ExecutionsPage component.
Move to folder pages
Replace Bootstrap with Mui
Replace Fortawesome with Mui icons
palagdan added a commit that referenced this issue Nov 1, 2024
@palagdan
[#62] Refactor Vocabulary Constants
04b4433 
Move vocabulary constants from Rest file to the separate vocabulary file.
palagdan added a commit that referenced this issue Nov 1, 2024
@palagdan
[#62] Refactor API REST
05da47e 
Install Axios for API calls.
Refactor the Rest module.
Define API URL constants.
Implement fetchers for Axios calls.
palagdan added a commit that referenced this issue Nov 1, 2024
@palagdan
[#62] Remove unused DynamicPage.jsx
96b700a
palagdan added a commit that referenced this issue Nov 1, 2024
@palagdan
[#62] Refactor Dagre constants to Vocabulary
bb79438
@palagdan palagdan mentioned this issue Nov 1, 2024
Closed
@blcham
Copy link
Contributor Author

blcham commented Nov 2, 2024 •
edited
Loading

@palagdan, I added new sub-issues to this issue and prioritized them. We should split this issue into separate issues (it is enough to do it incrementally, i.e. pick only the most important ones at the time and separate them into an issue)

This was referenced Nov 2, 2024
Merged
Closed
@palagdan palagdan mentioned this issue Nov 5, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@palagdan palagdan

Labels
refactoring
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@blcham @LaChope @palagdan