Skip to content

Conversation

@Aarekaz
Copy link
Owner

@Aarekaz Aarekaz commented Dec 4, 2025

Summary

Added comprehensive documentation for API, contributing, and keyboard shortcuts. Implemented Admin Dashboard, Receipt Scanning with HEIC/HEIF support, and enhanced keyboard shortcuts with a help dialog. Integrated PostHog Analytics and fixed keyboard shortcut input conflicts.

Changes Made

  • New documentation files: API_REFERENCE.md, CONTRIBUTING.md, KEYBOARD_SHORTCUTS.md.
  • Updated README.md with links to new documentation.
  • Implemented Admin Dashboard for bill management and statistics.
  • Added Receipt Scanning API with OCR support for various image formats, including HEIC/HEIF.
  • Enhanced keyboard shortcuts (N, P, C, S, ?) and introduced a dedicated help dialog.
  • Integrated PostHog Analytics for user behavior tracking.
  • Fixed keyboard shortcuts triggering while typing in input fields.
  • Improved Bill ID user experience.
  • Cleaned up todo.md by removing completed tasks.

Testing

  • All existing tests pass
  • New tests added for new functionality
  • Manual testing completed

Screenshots (if applicable)

[Add screenshots here]

Related Issues

Closes #ca525212-8581-4ffd-bd22-84bcb44331ea


Want me to make any changes? Add a review or comment with @tembo and i'll get back to work!

tembo.io app.tembo.io

Co-authored-by: Anurag <anuragdhungana5@gmail.com>
@vercel
Copy link
Contributor

vercel bot commented Dec 4, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
split-simple Error Error Dec 17, 2025 7:24pm

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 286 to 291
All admin endpoints require authentication via session token.

**Authentication Method:** Cookie-based sessions

**Required Cookie:** `admin_session_token`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Admin auth cookie name documented incorrectly

The Admin API section lists the required cookie as admin_session_token, but the authentication middleware actually sets and checks the admin_session cookie (lib/admin-auth.ts lines 6–36). Following the documented name will result in every admin request being rejected with 401 despite a valid session token, so the docs need to reflect the cookie name the API enforces.

Useful? React with 👍 / 👎.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tembo fix this

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed the cookie name in the documentation from admin_session_token to admin_session to match what's actually used in the authentication middleware.

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.

2 participants