Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

GQL-21: Create a GraphQL query for fetching available providers for a user #92

Merged
merged 35 commits into from
Feb 13, 2024

Conversation

tle1989
Copy link
Contributor

@tle1989 tle1989 commented Feb 2, 2024

Overview

What is the feature?

Create a GraphQl query for fetching available providers from CMR for a user

Please summarize the feature or fix.

Implement a GraphQL provider query that will get a list of providers from ACL endpoint CMR

What is the Solution?

So that on the React MMT App, a user can select a provider from their available list of providers from GraphQL

Summarize what you changed.
Add new type, new query, new tasks for acl

What areas of the application does this impact?

New React MMT

List impacted areas.

Dropdown box in React MMT where a user can select a provider from their available list of providers

Testing

Reproduction steps

  • Environment for testing:

  • GraphQL SIT

  • Collection to test with:

  1. Step 1
    In cmr-graphql

$CMR_ROOT_URL=http://cmr.sit.earthdata.nasa.gov/ MMT_ROOT_URL=http://mmt.sit.earthdata.nasa.gov/ DRAFT_MMT_ROOT_URL=http://localhost:3000/ npm start

  1. Step 2...
    Open Apollo Server
    Select ACL fields, and Params to query acl from Access-Control endpoint CMR

Attachments

Please include relevant screenshots or files that would be helpful in reviewing and verifying this change.

Checklist

  • [x ] I have added automated tests that prove my fix is effective or that my feature works
  • [x ] New and existing unit tests pass locally with my changes
  • [ x] I have performed a self-review of my own code
  • [ x] I have commented my code, particularly in hard-to-understand areas
  • [x ] I have made corresponding changes to the documentation
  • [x ] My changes generate no new warnings

@macrouch
Copy link
Contributor

macrouch commented Feb 8, 2024

This PR needs a lot of information added to the README.

I very confused about what is being added because the PR title says it is for adding a new query too fetch available providers, but nothing in the PR seems to be related to the providers.

Please fill out the PR template when opening a PR, it adds a lot of helpful context to the PR that needs to be there

@jceaser
Copy link

jceaser commented Feb 8, 2024

add a file defined at https://editorconfig.org to setup your tab and new line rules, most editors respect this file now.

@tle1989 tle1989 merged commit 1ed3c93 into main Feb 13, 2024
6 checks passed
@tle1989 tle1989 deleted the GQL-21 branch February 13, 2024 16:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants