-
Notifications
You must be signed in to change notification settings - Fork 939
Backfill Firestore Changelog #1275
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
# Unreleased | ||
|
||
# 0.8.2 | ||
- [fixed] Fixed an issue where native ES6 module loading was not working. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it worth mentioning webchannel in this at all?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just used the public release notes.
- [fixed] Fixed an issue where native ES6 module loading was not working. | ||
|
||
# 0.8.1 | ||
- [fixed] Fixed an issue where typings are created in the wrong location. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a way to rephrase this somehow to represent a user-visible difference in the SDK? This seems to be describing the cause of the issue rather than the symptom. Also "are" should be "were" to match the tense of "fixed"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leaving as is since this matches the public release notes.
BTW, this just copies the existing Firestore entries from https://firebase.google.com/support/release-notes/js |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh, okay then!
No description provided.