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

DEVDOCS-5923: [update] add customer access token #365

Merged
merged 26 commits into from
Jul 30, 2024
Merged

Conversation

bc-traciporter
Copy link
Collaborator

@bc-traciporter bc-traciporter commented Jul 8, 2024

DEVDOCS-5923

What changed?

Added customer access token to documentation

Release notes draft

Anything else?

ping {names}

@bc-traciporter bc-traciporter marked this pull request as ready for review July 15, 2024 15:10
@bc-traciporter bc-traciporter added the ready for review This PR is complete and awaiting a review. label Jul 15, 2024
Copy link
Contributor

@CNanninga CNanninga left a comment

Choose a reason for hiding this comment

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

In addition to in-line feedback: I'm wondering what language we might want to add to the Customer Impersonation Token section to frame how we expect devs to treat CITs now. I recommended moving the bullet points for why CATs are more secure to that section, so maybe that's sufficient. But are we going to flat-out say that CITs are considered a "legacy" approach now, or something similar? I'm not quie sure. I'd want Nathan or someone similarly invested in the product strategy to weigh in.

docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
@bc-traciporter
Copy link
Collaborator Author

@jamie-mckenzie can you provide me with a customer access token OAS file so I can put it here with the other ones?
https://developer.bigcommerce.com/docs/rest-authentication/tokens

Screenshot 2024-07-18 at 3 01 26 PM

docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/start/authentication/graphql-storefront.mdx Outdated Show resolved Hide resolved
docs/storefront/graphql/index.mdx Outdated Show resolved Hide resolved
@bc-traciporter
Copy link
Collaborator Author

Hi @kskomro I addressed your comments.

bookernath
bookernath previously approved these changes Jul 26, 2024
@bc-traciporter bc-traciporter merged commit df73de7 into main Jul 30, 2024
3 checks passed
@bc-traciporter bc-traciporter deleted the DEVDOCS-5923 branch July 30, 2024 18:32
becomevocal pushed a commit that referenced this pull request Aug 10, 2024
<!-- Ticket number or summary of work -->
# [DEVDOCS-5923]


## What changed?
Added customer access token to documentation

## Release notes draft
<!-- Provide an entry for the release notes using simple, conversational
language. Don't be too technical. Explain how the change will benefit
the merchant and link to the feature.

Examples:
* The newly-released [X feature] is now available to use. Now, you’ll be
able to [perform Y action].
* We're happy to announce [X feature], which can help you [perform Y
action].
* [X feature] helps you to create [Y response] using the [Z query
parameter]. Now, you can deliver [ex, localized shopping experiences for
your customers].
* Fixed a bug in the [X endpoint]. Now the [Y field] will appear when
you click [Z option]. -->
* 

## Anything else?
<!-- Add related PRs, salient notes, additional ticket numbers, etc. -->

ping {names}


[DEVDOCS-5923]:
https://bigcommercecloud.atlassian.net/browse/DEVDOCS-5923?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review This PR is complete and awaiting a review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants