#285-remove-cards-dashboard #359

Merged
b24johka merged 3 commits from #285-remove-cards-dashboard into team_1_week_2 2026-04-17 09:06:40 +00:00
Collaborator

Added the function to remove views/cards.

Added the function to remove views/cards.
cards can now be removed by pressing the "x" symbol
Collaborator

Review on #359

What's done

Reviewed issue #359

Tested on Windows 10 using Brave and LibreWolf

  • Delete card function works
  • App starts
    • The app starts correctly and new cards can be added.
    • The added cards can be removed as expected once the "Edit" mode is active and the x button is clicked.

Inspected following files

  • Dashboard.js
    • Indentation and styling looks good. Nothing needs correcting.
    • Comments
      • Comments explain the functionality clearly in an easy to understand way.
    • General code quality
      • Nothing to complain about. VSCode complains about a single unused import (React), but that is most likely for future use.
  • WindowsRebuild.bat
    • Inspected the file, but the entire file can be ignored as it was originally from another pull request, just pushed here on accident.

Conclusion

  • The feature works as intended. (Attached is a video of the feature being tested)
# Review on #359 ## What's done Reviewed issue #359 ### Tested on Windows 10 using Brave and LibreWolf + Delete card function works + App starts + The app starts correctly and new cards can be added. + The added cards can be removed as expected once the "Edit" mode is active and the **x** button is clicked. ### Inspected following files + Dashboard.js + Indentation and styling looks good. Nothing needs correcting. + Comments + Comments explain the functionality clearly in an easy to understand way. + General code quality + Nothing to complain about. VSCode complains about a single unused import (React), but that is most likely for future use. + WindowsRebuild.bat + Inspected the file, but the entire file can be ignored as it was originally from another pull request, just pushed here on accident. ## Conclusion + The feature works as intended. (Attached is a video of the feature being tested)
Collaborator

The review has been performed in a proper way. I will merge this now.

The review has been performed in a proper way. I will merge this now.
b24johka merged commit 36c0796bbd into team_1_week_2 2026-04-17 09:06:40 +00:00
b24johka deleted branch #285-remove-cards-dashboard 2026-04-17 09:06:49 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Andras/BoundlessFlowCampus2K!359
No description provided.