Skip to content
This repository was archived by the owner on Jan 22, 2026. It is now read-only.

Comments

feat: delete records#100

Merged
Pertempto merged 5 commits intomainfrom
91-delete-records
Apr 18, 2025
Merged

feat: delete records#100
Pertempto merged 5 commits intomainfrom
91-delete-records

Conversation

@JCampana98
Copy link
Collaborator

Closes #91

Changes

  • Removed AppBar handling from main.dart.
  • Added on hold selection logic to Records page.
  • Added multiple selection AppBar to Records page.
  • Added deletion logic for Records.
  • Moved Records related components to /record.

Testing

  • Go to the Records page.
  • Hold touch a record to select it.
  • Tap other records to add to selection.
  • AppBar should show the correct amount of selected records, a button to the left to stop selection mode and a button to the right to delete all selected items.
  • Clicking the delete button should prompt a confirmation dialog.
  • Records should be correctly deleted.

@JCampana98 JCampana98 requested a review from Pertempto April 18, 2025 14:49
@JCampana98 JCampana98 self-assigned this Apr 18, 2025
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Apr 18, 2025

Deploying karriba with  Cloudflare Pages  Cloudflare Pages

Latest commit: a058f21
Status: ✅  Deploy successful!
Preview URL: https://065b0e4e.karriba.pages.dev
Branch Preview URL: https://91-delete-records.karriba.pages.dev

View logs

@JCampana98
Copy link
Collaborator Author

Finished the changes 😄

@Pertempto
Copy link
Contributor

Very nice!! 👍🏽 👍🏽

@Pertempto Pertempto merged commit ad0ca03 into main Apr 18, 2025
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: delete records

2 participants