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
Copy file name to clipboardExpand all lines: README.md
+4-3Lines changed: 4 additions & 3 deletions
Original file line number
Diff line number
Diff line change
@@ -14,13 +14,14 @@ Run `docker-compose up` to start the containers. The dashboard is accessible at
14
14
15
15
-**Scan Service:** We could improve the logic in there to have less random results and maybe have something more sequential ( queued > scanning > result).
16
16
-**Auth:** An Auth service could be added to manage users and their scanning results.
17
-
-**Migrations:** Implementing a migration system for the models with sequelize. In the current release, there is an issue with the database access when trying to run migrations from the host.
17
+
-**Migrations:** Implementing a migration system for the models with sequelize.
18
18
19
19
### For the Dashboard
20
20
21
-
-**ScanDetails State Management:** Instead of loading all the data at once from the Dashboard, we could fetch only the required data for ScanList and then individually fetch the findings only when the corresponding ScanDetails component is diplayed (This is not relevant if the number of findings stays relatively low for each repo).
22
-
-**Error Handling:** The logic for handling axios request and the errors should implemented in a separate file
21
+
-**ScanDetails State Management:** Instead of loading all the data at once from the Dashboard, we could fetch only the required data for ScanList and then individually fetch the findings only when the corresponding ScanDetails component is displayed (This is not relevant if the number of findings stays relatively low for each repo).
22
+
-**Error Handling:** The logic for handling axios request and the errors should be implemented in a separate file
23
23
24
24
### For Docker
25
25
26
26
-**React HMR:** The HMR does not currently work when the dashboard is running in the container
27
+
-**DB Data persistence:** The data is erased from the DB when the container is stopped.
0 commit comments