Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KCAI request - Enhance QR Filter to Support External Barcode Scanner Input #1586

Closed
DonKoko opened this issue Jan 18, 2025 · 0 comments · Fixed by #1594
Closed

KCAI request - Enhance QR Filter to Support External Barcode Scanner Input #1586

DonKoko opened this issue Jan 18, 2025 · 0 comments · Fixed by #1594
Assignees
Labels

Comments

@DonKoko
Copy link
Contributor

DonKoko commented Jan 18, 2025

Description:
Extend the QR ID filter functionality to work with external barcode scanners by processing URL inputs into clean QR IDs.

Image

Technical Requirements:

  • Add URL parsing for shelf shortcodes
  • Strip URL components to extract QR ID
  • Support all filter operators (is, is not, contains, matches any, contains any)
  • Maintain compatibility with existing QR scanning functionality

Implementation Details:

  • Add URL detection and parsing logic
  • Create QR ID extraction function
  • Update filter input handling to process scanner input
  • Ensure compatibility across all search/filter fields

Testing Scenarios:

  1. Direct QR ID input
  2. Full URL input from scanner
  3. Various filter operator combinations
  4. Multiple sequential scans
  5. Edge cases (malformed URLs, invalid codes)
@DonKoko DonKoko converted this from a draft issue Jan 18, 2025
@DonKoko DonKoko self-assigned this Jan 18, 2025
@DonKoko DonKoko moved this from 🗓️ Planned to 🏗 In progress in 🗺️ Shelf Roadmap Jan 21, 2025
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in 🗺️ Shelf Roadmap Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant