Skip to content

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

Merged
merged 1 commit into from
Oct 2, 2018
Merged

Conversation

schmidt-sebastian
Copy link
Contributor

No description provided.

Copy link
Contributor

@wilhuff wilhuff left a 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.
Copy link
Contributor

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?

Copy link
Contributor Author

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.
Copy link
Contributor

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"

Copy link
Contributor Author

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.

@schmidt-sebastian
Copy link
Contributor Author

BTW, this just copies the existing Firestore entries from https://firebase.google.com/support/release-notes/js

Copy link
Contributor

@mikelehen mikelehen left a comment

Choose a reason for hiding this comment

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

Ahh, okay then!

@mikelehen mikelehen removed their assignment Oct 2, 2018
@schmidt-sebastian schmidt-sebastian merged commit f2cfd21 into master Oct 2, 2018
@schmidt-sebastian schmidt-sebastian deleted the schmidt-sebastian-patch-1 branch October 15, 2018 17:25
@firebase firebase locked and limited conversation to collaborators Oct 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants