Skip to content

BI-1872 - Generate Access Token for using BrAPI with DeltaBreed#327

Merged
timparsons merged 1 commit intodevelopfrom
feature/BI-1872
Sep 5, 2023
Merged

BI-1872 - Generate Access Token for using BrAPI with DeltaBreed#327
timparsons merged 1 commit intodevelopfrom
feature/BI-1872

Conversation

@timparsons
Copy link
Member

@timparsons timparsons commented Aug 30, 2023

Description

Story: https://breedinginsight.atlassian.net/browse/BI-1872

  • Updated the BrAPI page to use card components
  • Added a card for generating an API token

Dependencies

bi-api/feature/BI-1872

Testing

A)

  1. Log in to DeltaBreed
  2. Navigate to a program
  3. Navigate to the "BrAPI" page via the menu
  4. click on "Generate Access Token"
  5. Ensure that a new access token is generated and displayed with the expiration timestamp

B)

  1. Generate an access token (test A)
  2. Using that access token, make a REST call (via Insomnia/Postman):
    • header: Authorization and value of Bearer <access token>
    • URL: <program BrAPI URL>/germplasm (grab the BrAPI base URL on the BrAPI page in DeltaBreed)
  3. Ensure that the rest call returns back the germplasm for that program

Checklist:

  • I have performed a self-review of my own code
  • I have tested my code and ensured it meets the acceptance criteria of the story
  • I have create/modified unit tests to cover this change
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to documentation
  • I have run TAF: https://github.com/Breeding-Insight/taf/actions/runs/6027686471

Copy link
Contributor

@mlm483 mlm483 left a comment

Choose a reason for hiding this comment

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

Works, approved.

@timparsons timparsons merged commit ff89b80 into develop Sep 5, 2023
@timparsons timparsons deleted the feature/BI-1872 branch September 5, 2023 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants