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

allow login to AppSync style backends too #35

Merged
merged 1 commit into from
Jul 17, 2024
Merged

Conversation

fredex42
Copy link
Collaborator

What does this change?

Adds an option to select the API path, allowing AppSync style URIs (/graphql) as well as conventional ones (/query)

How to test

  • Use the 'GraphQL endpoint' param from the AppSync api "fronts-graphql-test" in the CAPI account
  • Use an API key from the configuration of that API
  • Select the "[Appsync] api type and it should work. If you don't you'll see an error along the lines "unexpected operation"

How can we measure success?

Able to talk to AppSync

Have we considered potential risks?

n/a

Images

Screenshot from 2024-05-24 15-11-51

Copy link

Unit Test Results

0 tests  ±0   0 ✔️ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ±0 

Results for commit 01d33f3. ± Comparison against base commit dd1d0d1.

@fredex42 fredex42 merged commit 2f13ba3 into main Jul 17, 2024
6 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.

1 participant