Skip to content

Add indexedDB to webworker export #3630

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
wants to merge 89 commits into
base: auth-next
Choose a base branch
from
Open

Conversation

sam-gc
Copy link
Contributor

@sam-gc sam-gc commented Aug 13, 2020

No description provided.

sam-gc and others added 30 commits August 10, 2020 12:55
Initial package structure--copied from packages/template, but with tests working
Add mock fetch library for tests
* Add core types & error map

* [AUTOMATED]: Prettier Code Styling

* [AUTOMATED]: License Headers

* User should be an interface for now

we can make an implementation class later
* User should be an interface for now

we can make an implementation class later

* Add API call to signUp

* [AUTOMATED]: Prettier Code Styling

* [AUTOMATED]: License Headers

* Update tests to test a little more

* [AUTOMATED]: Prettier Code Styling
* Initial user object implementation
* Consolidate mock auth / add user test helper

* [AUTOMATED]: Prettier Code Styling

* Fix tests
* User should be an interface for now

we can make an implementation class later

* Add API call to signUp

* [AUTOMATED]: Prettier Code Styling

* Update tests to test a little more

* Add remaining API methods to auth-next

* [AUTOMATED]: Prettier Code Styling

* [AUTOMATED]: License Headers

* Add more tests to account API methods

* Pass in SDK version & correctly send GET request params

* [AUTOMATED]: Prettier Code Styling

* [AUTOMATED]: Prettier Code Styling

* Fix import ordering

* Minor formatting change

* [AUTOMATED]: Prettier Code Styling
* Add support for API timeouts to auth-next

* PR feedback

* [AUTOMATED]: Prettier Code Styling
* Add version helper to auth-next

* [AUTOMATED]: Prettier Code Styling

* PR feedback
* Add fetchSignInMethodsForEmail to auth-next

* [AUTOMATED]: Prettier Code Styling

* PR Feedback

* [AUTOMATED]: Prettier Code Styling
* Add sendEmailVerification to auth-next

* [AUTOMATED]: Prettier Code Styling

* Add missing test

* Remove GetOobCodeRequestType in favor of Operation
* Add sendEmailVerification to auth-next

* [AUTOMATED]: Prettier Code Styling

* Remove GetOobCodeRequestType in favor of Operation

* Add password reset methods to auth-next

* [AUTOMATED]: Prettier Code Styling

* PR Feedback

* [AUTOMATED]: Prettier Code Styling

* [AUTOMATED]: Prettier Code Styling

* Resolve merge conflicts

* [AUTOMATED]: Prettier Code Styling
* Use queryString from firebase/util instead of homebrew one

* [AUTOMATED]: Prettier Code Styling

* PR Feedback

* [AUTOMATED]: Prettier Code Styling
* Add conditional delays to auth-next

* [AUTOMATED]: Prettier Code Styling

* Use typescript asserts keyword for typesafe assertions

* [AUTOMATED]: Prettier Code Styling

* Rebase conflicts & PR feedback

* More PR Feedback

* Strip debug asserts from prod builds

* [AUTOMATED]: Prettier Code Styling

* [AUTOMATED]: API Reports

* Fix logic in assertion

* Revert merge artifacts

* PR Feedback
* Add sendSignInWithEmail links

* [AUTOMATED]: Prettier Code Styling

* PR Feedback

* PR Feedback
* Standardize auth-next style

- prefix directory for test blocks
- use kebab style for test values to make them visually distinct from
keys
- use '#' to prefix instance methods and '.' for class methods
- prefix all private exported methods with '_'

* [AUTOMATED]: Prettier Code Styling

* Two more cases I missed
Added React Native persistence class
* Update internal types to work with external types

* [AUTOMATED]: Prettier Code Styling

* [AUTOMATED]: License Headers

* PR feedback

* [AUTOMATED]: Prettier Code Styling
…#3078)

* Further refactoring of auth types. Add public exports to src/index.ts

* Add app-exp to devDependencies

* PR feedback
* user.delete() implementation

* [AUTOMATED]: Prettier Code Styling

* PR feedback

* PR feedback
* Add signInWithCredential to auth-next

* [AUTOMATED]: Prettier Code Styling

* Add conditional delays to auth-next (#2934)

* Add conditional delays to auth-next

* [AUTOMATED]: Prettier Code Styling

* Use typescript asserts keyword for typesafe assertions

* [AUTOMATED]: Prettier Code Styling

* Rebase conflicts & PR feedback

* More PR Feedback

* Strip debug asserts from prod builds

* [AUTOMATED]: Prettier Code Styling

* [AUTOMATED]: API Reports

* Fix logic in assertion

* Revert merge artifacts

* PR Feedback

* Cleanup & PR Feedback

* [AUTOMATED]: Prettier Code Styling

* [AUTOMATED]: License Headers

* Add more tests

* [AUTOMATED]: Prettier Code Styling

* Rebase conflicts

* Cleanup type exports

* PR feedback

* Declare abstract class abstractly
sam-gc and others added 15 commits August 10, 2020 12:55
* Add a standard OAuthCredential and a GoogleAuthProvider

* Formatting

* Signature updates

* Formatting

* Update w/ demo

* PR feedback plus a bugfix

* Formatting
* Add the rest of the OAuth providers

* Formatting

* Add to demo
* Add error handling for account-exists-with-different-credentials edgecase

* Tests

* Formatting
* First pass at polyfill

* PR Feedback
* Copy demo code over to auth-compat-layer, fix our build scripts

* Add more functionality to compat layer

* PR Feedback

* PR Feedback
* Further refactoring of auth types. Add public exports to src/index.ts (#3078)

* Further refactoring of auth types. Add public exports to src/index.ts

* Add app-exp to devDependencies

* PR feedback

* Fix more edge cases around auth-compat-exp polyfill

* Add uglified polyfill build for size comparison

* PR feedback

* Set ClientPlatform correctily in auth-exp and auth-compat-exp

Co-authored-by: Sam Horlbeck Olsen <sam.horlsen@gmail.com>
* Add assertTypes util

* Add assertTypes function
* Clean up our to/from JSON implementations

- Remove anonymous credential/provider since they're not exposed and
don't do anything
- rename toPlainObject on User to toJSON per API spec
- relax restrictions around ProviderId & SignInMethod typing since they
 can sometimes have values outside of our enum

* PR feedback and more cleanup

* Rejigger our error type checking to allow app-less errors

* PR feedback
@changeset-bot
Copy link

changeset-bot bot commented Aug 13, 2020

💥 No Changeset

Latest commit: a16df27

Merging this PR will not cause any packages to be released. If these changes should not cause updates to packages in this repo, this is fine 🙂

If these changes should be published to npm, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@google-oss-bot
Copy link
Contributor

Binary Size Report

Affected SDKs

No changes between base commit (55d3c26) and head commit (c13ca50).

Test Logs

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.

5 participants