Skip to content

Adding use-case#188

Merged
meker12 merged 20 commits intoccdm-early-accessfrom
em_ccdm-usecase
Feb 6, 2025
Merged

Adding use-case#188
meker12 merged 20 commits intoccdm-early-accessfrom
em_ccdm-usecase

Conversation

@erikmarr
Copy link
Contributor

Purpose of this pull request

This pull request (PR) adds a new topic that walks the user through an end-to-end example of using CCDM.

Affected pages

@erikmarr erikmarr self-assigned this Jan 22, 2025
@erikmarr erikmarr requested a review from meker12 January 22, 2025 22:19
Comment on lines 300 to 304
- **`AC-Channel-Id`: `channelId-1`**
- `AC-Policy-State`: `AC-Policy-Country`
- `X-Api-Key`: `search_gql`
- **`AC-Policy-Country`: `USA`**
- **`AC-Policy-Brand`: `Brand A`**
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do some of the headers have ** delimiters?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@meker12 I was trying to visually call out those headers that are unique to CCDM. But maybe the note I put at the end of this list is good enough?

Copy link
Contributor

@meker12 meker12 Jan 24, 2025

Choose a reason for hiding this comment

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

Oh I see what you mean about trying to callout the headers --- maybe it would be better to mentiont that the brand and location are passed as headers and include the header in the section overview description instead of just Brand A (AC-Policy-Brand: A)

}
}
```

Copy link
Contributor

Choose a reason for hiding this comment

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

Same comment about having one section -- Create policies with the overview statement of what is happening followed by the two code samples in the same section.

Comment on lines 300 to 304
- **`AC-Channel-Id`: `channelId-1`**
- `AC-Policy-State`: `AC-Policy-Country`
- `X-Api-Key`: `search_gql`
- **`AC-Policy-Country`: `USA`**
- **`AC-Policy-Brand`: `Brand A`**
Copy link
Contributor

@meker12 meker12 Jan 24, 2025

Choose a reason for hiding this comment

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

Oh I see what you mean about trying to callout the headers --- maybe it would be better to mentiont that the brand and location are passed as headers and include the header in the section overview description instead of just Brand A (AC-Policy-Brand: A)


>[!INFO]
>
>In the above headers, `AC-Channel-Id`, `AC-Policy-Country`, and `AC-Policy-Brand` are concepts unique to CCDM.
Copy link
Contributor

Choose a reason for hiding this comment

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

If you update the description, you could link to the Header documentation that explains what the Headers are instead of using the note. Maybe you can call them CCDM headers -- (link: https://developer-stage.adobe.com/commerce/services/composable-catalog/storefront-services/using-the-api/#headers)

}
```

### Storefront API call – 2: Retrieve SKU for Brand B and location UK
Copy link
Contributor

@meker12 meker12 Jan 24, 2025

Choose a reason for hiding this comment

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

I think we should include the code for the entire data ingestion operations the way we do for these GraphQL queries. I'll get the code.

@meker12 meker12 merged commit 3533f3d into ccdm-early-access Feb 6, 2025
9 checks passed
@meker12 meker12 deleted the em_ccdm-usecase branch February 6, 2025 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants