Skip to content

docs: update landing auth example#641

Merged
arferreira merged 3 commits into
rapina-rs:mainfrom
Kkkakania:docs/update-public-example-current-user
Jun 20, 2026
Merged

docs: update landing auth example#641
arferreira merged 3 commits into
rapina-rs:mainfrom
Kkkakania:docs/update-public-example-current-user

Conversation

@Kkkakania

@Kkkakania Kkkakania commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Closes #640.

Summary

  • Update the landing page "Just Add #[public]" example to use the current CurrentUser extractor instead of the outdated Claims handler parameter.
  • Adjust the surrounding copy and comment to match the current extractor docs.

Validation

  • rg verified Claims / claims.sub no longer appear in docs/templates/index.html
  • git diff --check passed

Note: I did not run the Zola docs build locally because zola is not installed in this environment.

@Kkkakania Kkkakania requested a review from arferreira as a code owner June 17, 2026 04:11
@github-actions

Copy link
Copy Markdown

Thanks for your first PR! A maintainer will review it shortly.

Make sure CI passes and take a look at our contributing guide if you haven't already.

@github-actions github-actions Bot added the area: docs Documentation site label Jun 17, 2026
@arferreira arferreira requested review from richasdev and removed request for arferreira June 17, 2026 10:25
Comment thread docs/templates/index.html Outdated
Comment thread docs/templates/index.html

@ShiraiEd ShiraiEd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hi, great first contribution!

I have left 2 comments as suggestions that I think that would improve this a little more,
then once addressed, it will look got to merge to me

@Kkkakania

Copy link
Copy Markdown
Contributor Author

Updated the branch to address both review suggestions:

  • clarified the text to say the handler accesses the current user's data
  • expanded the inline code comment to describe the authenticated user id lookup and JSON response

@ShiraiEd ShiraiEd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@arferreira arferreira merged commit 7273071 into rapina-rs:main Jun 20, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: docs Documentation site

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Good First Issue]: Update landing page code example

3 participants