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: Amplify JS release #13212

Merged
merged 13 commits into from
Apr 2, 2024
Merged

release: Amplify JS release #13212

merged 13 commits into from
Apr 2, 2024

Conversation

Samaritan1011001
Copy link
Contributor

Description of changes

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.

jimblanc and others added 13 commits March 26, 2024 08:31
chore(api-graphql): improve error handling

  - Use GraphApiError to create errors to be thrown
    * error message field remained the same as before
    * added recoverySuggestion field for each error case
  - Created createGraphQLResultWithError utility for rewrapping error into GraphQLResult format
* fix(auth): signup should always return a userId

* add unit tests

* Update packages/auth/__tests__/providers/cognito/signUp.test.ts

Co-authored-by: Chris F <5827964+cshfang@users.noreply.github.com>

* address feedback

---------

Co-authored-by: Ashwin Kumar <ashwsrir@amazon.com>
Co-authored-by: Chris F <5827964+cshfang@users.noreply.github.com>
… core (#13108)

* chore(repo): setup the base for lint __tests__ codebase

* chore(adapter-nextjs): run yarn lint:fix

* chore(adapter-nextjs): manual fix linter reported erros

* chore(aws-amplify): run yarn lint:fix
 Please enter the commit message for your changes. Lines starting

* chore(aws-amplify): add camelcase var name exception: phone_number

* chore(core): run yarn lint:fix

* chore(core): manual fix linter reported errors under __tests__

* apply suggestions

* turn off node callback function convention check

* do not mock console functions globaly but noopify them

* apply suggestions

* Revert "do not mock console functions globaly but noopify them"

This reverts commit 913b11e.

* refactor a no-op function

* remove // eslint-disable-next-line no-new
Revert "fix(api-graphql): default selection set regression (#13185)"

This reverts commit 873a6f4.
@Samaritan1011001 Samaritan1011001 merged commit e7317d6 into release Apr 2, 2024
307 checks passed
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.

None yet

7 participants