You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It's really nice that you managed to bundle the entire agent in one file, 600 lines long lmao. While experienced users may be able to navigate around it, new contributors may find it difficult to add features. I believe a better approach would be to split everything (agent, hub, site) into multiple files so it's easier to navigate. For example the UI could be split into multiple components each component doing a specific thing. Additionally the agent can be again split into 2-3 files for the data fetching, data sending etc. The hub can stay as is I believe since it's relatively small. Additionally it could be useful to add more comments : ). That's definitely a future idea but it would be a solid refactor. I can start adding some comments here and there if you like.
The text was updated successfully, but these errors were encountered:
Hello,
It's really nice that you managed to bundle the entire agent in one file, 600 lines long lmao. While experienced users may be able to navigate around it, new contributors may find it difficult to add features. I believe a better approach would be to split everything (agent, hub, site) into multiple files so it's easier to navigate. For example the UI could be split into multiple components each component doing a specific thing. Additionally the agent can be again split into 2-3 files for the data fetching, data sending etc. The hub can stay as is I believe since it's relatively small. Additionally it could be useful to add more comments : ). That's definitely a future idea but it would be a solid refactor. I can start adding some comments here and there if you like.
The text was updated successfully, but these errors were encountered: