Skip to content

README.md misused variable #3344

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
KONRADS098 opened this issue May 1, 2023 · 1 comment
Open

README.md misused variable #3344

KONRADS098 opened this issue May 1, 2023 · 1 comment

Comments

@KONRADS098
Copy link
Contributor

Head to https://github.com/angular/angularfire/blob/master/README.md

Version info

Angular:
N/A
Firebase:
N/A
AngularFire:
N/A
Other (e.g. Ionic/Cordova, Node, browser, operating system):
N/A

How to reproduce these conditions

Failing test unit, Stackblitz demonstrating the problem
N/A
Steps to set up and reproduce
N/A
Sample data and security rules
N/A

Debug output

Errors in the JavaScript console
N/A
Output from firebase.database().enableLogging(true);
N/A
Screenshots
N/A

Related Issues

#3343

Expected behavior

The declared retrieved collection itemCollection is to be used by passing it into the collectionData method as a parameter.

constructor() {
    const itemCollection = collection(this.firestore, 'items');
    this.item$ = collectionData(itemCollection);
}

Actual behavior

The declared retrieved collection itemCollection is not being used, instead an unexisting collection collection is being used.

constructor() {
    const itemCollection = collection(this.firestore, 'items');
    this.item$ = collectionData(collection);
}
@KONRADS098 KONRADS098 changed the title Update README.md Typo README.md misused variable May 2, 2023
@KONRADS098
Copy link
Contributor Author

Fixed!

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

No branches or pull requests

1 participant