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

Added note on write_suites API access token scope to README #344

Merged
merged 2 commits into from Aug 2, 2023

Conversation

james2791
Copy link
Contributor

With the addition of the Test Suite resource in v0.22.0, one's API access token that is used to interact with the provider needs to additionally have the write_suites scope for said API token to create/update and delete suites (interaction with the REST API)

@jradtilbrook
Copy link
Member

@james2791 I think this is documented in a few places that will need updating as well. And to confirm, it doesn't need read_suites because it uses GraphQL for that side of things right?

@james2791
Copy link
Contributor Author

james2791 commented Aug 2, 2023

@james2791 I think this is documented in a few places that will need updating as well. And to confirm, it doesn't need read_suites because it uses GraphQL for that side of things right?

Just added to the other spots - correct too: the Read of the Test suite resource being node based, this more for the C-U-D ops otherwise that interacts with REST

@james2791 james2791 merged commit bee253e into main Aug 2, 2023
1 check passed
@james2791 james2791 deleted the js/Test-Suite-Readme-API-Scope branch August 2, 2023 01:36
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

2 participants