Skip to content

Update auth.md change user$ = user(auth); to user$ = user(this.auth);#3640

Merged
armando-navarro merged 3 commits into
angular:mainfrom
arolleaguekeng:patch-1
Jul 8, 2026
Merged

Update auth.md change user$ = user(auth); to user$ = user(this.auth);#3640
armando-navarro merged 3 commits into
angular:mainfrom
arolleaguekeng:patch-1

Conversation

@arolleaguekeng

Copy link
Copy Markdown
Contributor

Checklist

Description

This PR fixes an issue in the AngularFire documentation where the auth instance was incorrectly referenced as user(auth), instead of using the injected instance (this.auth).

Description

Code sample

This PR fixes an issue in the AngularFire documentation where the auth instance is incorrectly referenced in user(auth). The correct approach is to use the injected auth instance (this.auth) instead. The updated code ensures proper dependency injection and aligns with best practices in Angular.

@armando-navarro armando-navarro 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.

Thank you for this correction.

@armando-navarro armando-navarro merged commit 3f7fe29 into angular:main Jul 8, 2026
24 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