Skip to content

Feat/e2e remote badge delete#12

Open
AdWav wants to merge 3 commits into
developfrom
feat/e2e-remote-badge-delete
Open

Feat/e2e remote badge delete#12
AdWav wants to merge 3 commits into
developfrom
feat/e2e-remote-badge-delete

Conversation

@AdWav
Copy link
Copy Markdown
Contributor

@AdWav AdWav commented Apr 22, 2026

Summary

  • Fix log/history retrieval to query by resolved user_id only instead of mixing badge_id and user_id filters.
  • Add defensive early returns when badge-to-user resolution fails (None or -1) to avoid invalid queries.
  • Reorganize README documentation headings to separate global documentation notes from E2E documentation.
  • Update docs/DATABASE.md to align database documentation with the E2E deletion/synchronization workflow.
  • Add explicit server/client table mapping for E2E validation (event_type, event_log_sync, *_sync, *_write), plus SQL verification guidance tied to E2E logs.

Why

  • The query changes prevent inconsistent history results caused by mixed identity filters and make behavior safer when user resolution fails.
  • The README update improves navigation between general project documentation and E2E-specific procedures.
  • The database documentation now better reflects the real end-to-end deletion lifecycle (soft/hard delete, event import, and processing state), improving traceability and debugging.

Test Plan

  • Run E2E tests.
  • Validate user info lookup and 5-last-logs behavior with a valid badge.
  • Validate behavior with an unknown badge (no crash, empty data returned).
  • Manually verify README section structure renders correctly.
  • Review docs/DATABASE.md and cross-check E2E table/event semantics against tests/E2E/README.md.
  • Confirm documented SQL checks match snapshots in tests/E2E/logs/E2E_logs.txt.

AdWav added 3 commits April 22, 2026 11:56
Add event-log synchronization and local cleanup logic so remote user and badge hard deletes are reflected before badge scans.
Include local E2E tooling and regression tests to validate remote-delete behavior end-to-end.
Prevent log/history lookups from mixing badge and user filters by querying with resolved user id only. Also add early returns for unknown users and reorganize README documentation sections.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant