Skip to content

fix: complete CardSnap rebranding - add screenshots and fix remaining issues#24

Closed
rprabhat wants to merge 4 commits into
mainfrom
fix/final-cardsnap-cleanup
Closed

fix: complete CardSnap rebranding - add screenshots and fix remaining issues#24
rprabhat wants to merge 4 commits into
mainfrom
fix/final-cardsnap-cleanup

Conversation

@rprabhat

@rprabhat rprabhat commented Apr 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

Complete the CardSnap rebranding by addressing all Oracle verification findings.

Changes

  • Add 4 SVG app screenshots to docs/images/screenshots/:
    • scan-screen.svg - Camera scan screen
    • review-contact.svg - Contact review/edit screen
    • contacts-list.svg - Contacts list
    • settings.svg - Settings screen
  • Fix PRIVACY_POLICY.md - Rewritten for native Kotlin Android (removed React Native and iOS references)
  • Remove React Native from Gradle - All com.facebook.react, react-native, and node_modules references removed
  • Update deep link scheme - cardscanner://cardsnap://
  • Fix package.json - Keyword card-scannercard-snap

AI Disclosure

  • AI-Generated: Yes
  • Model: Qwen 3.6 Plus
  • Platform: OpenCode (OhMyOpenCode)
  • Human Oversight: fully reviewed

- Add GitHub Pages landing page (docs/index.html)
- Add 4 SVG app screenshots (scan, review, contacts, settings)
- Update .gitignore to allow docs/index.html and screenshots
- Fix PRIVACY_POLICY.md for native Kotlin Android (remove RN/iOS refs)
- Remove all React Native references from Gradle configs
- Ensure deep link scheme is cardsnap:// (not cardscanner://)
- Fix package.json keyword to card-snap
- Update CODE_OF_CONDUCT.md to Contributor Covenant v2.1

Co-Authored-By: Claude <noreply@anthropic.com>
@rprabhat rprabhat force-pushed the fix/final-cardsnap-cleanup branch from 7cea79b to 70ebd12 Compare April 5, 2026 12:20
@rprabhat

rprabhat commented Apr 5, 2026

Copy link
Copy Markdown
Collaborator Author

Closing - gradle conversion from Kotlin DSL to Groovy DSL has fundamental issues requiring RN/Android expertise to resolve. This is a rebranding PR, not a Sensible standards compliance issue.

@rprabhat rprabhat closed this Apr 5, 2026
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