8000 feat(auth): remember when someone has seen the welcome tour by adeelahmad · Pull Request #152 · adeelahmad/lense · GitHub
[go: up one dir, main page]

Skip to content

feat(auth): remember when someone has seen the welcome tour - #152

Merged
adeelahmad merged 1 commit into
mainfrom
claude/project-thread-2sexnc
Oct 5, 2026
Merged

adeelahmad merged 1 commit into
mainfrom
claude/project-thread-2sexnc

Conversation

@adeelahmad
Copy link
Copy Markdown
Owner

Requested by Adeel · project thread

Description

Nothing changes on screen yet. This is the first of two PRs for a first-sign-in welcome tour that leads new people to the assistant and the charts (Reports).

  • GET /auth/me now answers toured_at: empty until the person finishes or skips the tour.
  • POST /auth/me/tour records it. Repeating it keeps the first time; API tokens with read scope get 403.
  • A tour-seen upgrade step marks everyone who had already signed in, so existing users never see the tour. Only people who sign in for the first time after the upgrade do.
  • OpenAPI schema and client regenerated with make openapi.

Motivation and Context

Keeping the flag on the account means the tour shows once per person, not once per browser.

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist:

  • The title is a Conventional Commit.
  • My code follows the code style of this project (ruff check and format pass).
  • I added or updated tests (test_welcome_tour_opens_until_finished, test_people_who_signed_in_before_the_tour_skip_it; migration lists updated).

Locally the full backend suite passes except test_office_and_text_files_become_documents, which fails the same way on main in this sandbox because LibreOffice can't load the file there.

🤖 Generated with Claude Code

https://claude.ai/code/session_01P2JaBE5twABdk56QrYovMi


Generated by Claude Code

/auth/me answers toured_at (None until the tour is finished or skipped)
and POST /auth/me/tour records it once. A tour-seen upgrade step marks
everyone who had already signed in, so only new people get the tour.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01P2JaBE5twABdk56QrYovMi
@adeelahmad adeelahmad self-assigned this Oct 5, 2026
@adeelahmad
adeelahmad merged commit 627be73 into main Oct 5, 2026
3 of 5 checks passed
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

0