Skip to content
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

release(required): Amplify JS release #11811

Merged
merged 8 commits into from
Aug 16, 2023
Merged

release(required): Amplify JS release #11811

merged 8 commits into from
Aug 16, 2023

Conversation

kvramyasri7
Copy link
Contributor

Description of changes

amplify-js release to resolve a regression

Issue #, if available

Description of how you validated changes

Checklist

  • PR description included
  • yarn test passes
  • Tests are changed or added
  • Relevant documentation is changed or added (and PR referenced)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

HuiSF and others added 8 commits August 11, 2023 11:09
* chore: update xcode version to 13.4.1

* chore: comment integ_rn_ios_push_notifications on cci

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>
* chore: copy Auth code to InternalAuth

* feat: add customUserAgentDetails param to InternalAuth methods

* feat: make AuthClass extend InternalAuthClass, override functions with extra param

* build: export auth internals from /internals scope

* fix: changes signUp back to method type instead of prop so Auth can extend the method

* chore: update auth lint settings to allow class arrow functions to end in semi-colon

* fix: add overloaded method signature back in - unrelated change

* test: fix auth tests by mocking InternalAuthClass instead of AuthClass

* chore: change arrow methods to normal functions

* fix: remove async from overridden async functions

* chore: increase bundle size limits

* chore: publish feature branch for testing

* chore: port code from CognitoUser and CognitoUserPool to internals

* chore: delete InternalCognitoUserPool - separate PR

* chore: update imports and class name

* feat: add userAgentValue param to cognito client api's

* feat: add userAgentValue param to InternalCognitoUser api's, pass to client

* feat: make CognitoUser extend InternalCognitoUser and override api's

* test: update tests to work with additional param, minor fix

* chore: clean up unused imports and const's

* feat: add missing userAgentValue param

* chore: add new param to documentation comments

* build: export InternalCognitoUser from internals scope

* fix: remove default from InternalCognitoUser to fix scoped export

* fix: fix typo causing signOut failure

* build: correct index.d.ts files with changes

* test: revert auth tests

* chore: increase bundle size limits

* chore: increase bundle size limits

* chore: port CognitoUserPool over to InternalCognitoUserPool

* chore: restore CognitoUserPool

* feat: add userAgentValue parameter to signUp

* feat: update class name and imports

* feat: make CognitoUserPool extend internal and override signUp

* feat: export InternalCognitoUserPool from internals index

* feat: send userAgentValue to client service call

* test: fix internalsIndex test

* build: export InternalCognitoUserPool from internals scope

* chore: update CognitoUserPool type to extend internal

* chore: increase bundle size limits

* feat: add utils for auth user agent

* feat: add/uncomment AuthActions

* feat: pass userAgentValue to OAuth

* feat: send user agent details through cognito

* test: update test to user InternalCognitoUserPool

* test: update test to user internal cognito classes

* fix: send user agent with auth details instead of received

* test: mock cognito internals in auth-attribute-test

* test: mock cognito internals in auth-federation-unit-tests

* test: mock cognito internals in auth-refresh-token-test

* test: mock cognito internals in hosted-ui.test

* test: mock cognito internals in totp-unit-test

* fix: add in missed user agent details

* fix: pass user agent details to private implementations

* fix: send user agent string instead of object

* fix: accidental replacement of currentUserPoolUser

* test: mock new private methods

* test: mock private method

* chore: send undefined instead of null for userAgentValue:

* test: account for added param in OAuth

* chore: increase bundle size limits

* chore: remove publishing of preid release

* chore: increase bundle size limit

* chore: remove comment

* fix: send undefined clientMetadata before useragentvalue

* fix: use InternalCognitoUser as underlying instance

* fix: send userAgentValue through fetchUser

* fix: send userAgentValue through _getUserData

* test: fix tests to expect userAgentValue

* fix: add missing param and AuthAction for currentCredentials

* chore: increase bundle size limits
* chore: fix actions amplify-js build caching

* chore: fix actions amplify-js build caching

* chore: increase timeout for datastore tests

---------

Co-authored-by: Sridhar <ashwsrir@amazon.com>
Co-authored-by: israx <70438514+israx@users.noreply.github.com>
* feat: send pubsub user agent details to auth calls

* test: test user agent getting to auth
@jimblanc jimblanc marked this pull request as ready for review August 16, 2023 19:41
@jimblanc jimblanc requested review from a team as code owners August 16, 2023 19:41
@jimblanc jimblanc merged commit e3ad861 into release Aug 16, 2023
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.

6 participants