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

fix(api): issue 12756 by exporting CONNECTION_STATE_CHANGE as a value #12757

Conversation

timheilman
Copy link
Contributor

rather than as a type

Description of changes

It looks like a simple oversight, that CONNECTION_STATE_CHANGE was included in an export type statement rather than in a plain export statement.

Issue #, if available

12756

Description of how you validated changes

Writing a failing unit test, writing the fix, seeing the unit test pass.

Checklist

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

@timheilman timheilman requested review from a team as code owners December 27, 2023 18:30
@timheilman timheilman changed the title fix: issue 12756 by exporting CONNECTION_STATE_CHANGE as a value fix(api): issue 12756 by exporting CONNECTION_STATE_CHANGE as a value Dec 27, 2023
@cwomack cwomack self-assigned this Dec 27, 2023
@cwomack cwomack added GraphQL Related to GraphQL API issues amplify/js labels Dec 27, 2023
Copy link
Contributor

@iartemiev iartemiev left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the contribution!

@stocaaro stocaaro merged commit bedbdbd into aws-amplify:main Jan 2, 2024
30 checks passed
ashwinkumar6 added a commit that referenced this pull request Jan 2, 2024
* chore: enable e2e in-app test

* chore: correct integ sample path

* feat(ci): allow 0BSD license (#12761)

Co-authored-by: Ashwin Kumar <ashwsrir@amazon.com>

* fix(api): issue 12756 by exporting CONNECTION_STATE_CHANGE as a value (#12757)

fix: issue 12756 by exporting CONNECTION_STATE_CHANGE as a value

rather than as a type

Co-authored-by: Ivan Artemiev <29709626+iartemiev@users.noreply.github.com>

---------

Co-authored-by: ManojNB <manojnb95@gmail.com>
Co-authored-by: ManojNB <manojnb@amazon.com>
Co-authored-by: Ashwin Kumar <ashwsrir@amazon.com>
Co-authored-by: Tim <timheilman@users.noreply.github.com>
Co-authored-by: Ivan Artemiev <29709626+iartemiev@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
amplify/js GraphQL Related to GraphQL API issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants