Skip to content

DCAT-2416 Refactor navigation and editorial updates#253

Merged
meker12 merged 138 commits intoccdm-early-accessfrom
DCAT-2416-refactor-navigation
Jun 24, 2025
Merged

DCAT-2416 Refactor navigation and editorial updates#253
meker12 merged 138 commits intoccdm-early-accessfrom
DCAT-2416-refactor-navigation

Conversation

@meker12
Copy link
Copy Markdown
Contributor

@meker12 meker12 commented Jun 10, 2025

Purpose of this pull request

This PR refactors the top navigation for Merchandising Services and creates stand-alone API reference pages for the Data Ingestion REST API, and the Merchandising Services GraphQL API

  • Move authentication for Data ingestion API to it's own topic
    • Clarify that this auth is for direct, interactive access, and link to User Auth Implementation guide for more info
  • Change "Using topics" to getting started
  • Add API Reference header
  • Create a reference folder for stand-alone API reference pages
  • Update spectaql storefront metadata to remove CategoryView and CategoryViewInterface from the Merchandising Services API reference
  • Fix landing page to match what's on developer.adobe.com
  • Add reference folder with API reference pages
  • Editorial changes to remove references to the ACO Admin API and redirect to ACO UI.
  • Add edition tags

dshevtsov and others added 30 commits March 10, 2025 17:18
CCSAAS-866 & CCSAAS-1510 - REST intro and updating reference
Co-authored-by: Jeff Matthews <matthews.jeffery@gmail.com>
CCSAAS-796: Editorial reorganization for ACCS
keharper and others added 16 commits June 19, 2025 11:16
- Update navigation to point to the new reference pages
- Rename "storefront api" to "merchandising api" (part 1)
- Create reference folder for Merchandising API and Data Ingestion API
- Update navigation to add API reference sections
- Rename Storefront API to Merchandising API
- Improve overview for Merchandising API and Data Ingestion API
- Add get started, use cases, and troubleshooting sections for Merchandising API
- Refactor navigation to remove CS, LS, and PREX content
Fix metadata
@meker12 meker12 requested a review from jeff-matthews June 23, 2025 22:37
Copy link
Copy Markdown
Contributor

@jeff-matthews jeff-matthews left a comment

Choose a reason for hiding this comment

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

Two things stand out on cursory review:

  • The links to API Reference 404. Is that expected?
  • The edition: ee metadata key-value pair is no longer supported, which is why pages with that metadata are rendering the fallback yellow badge. You'll need to update that.

@meker12
Copy link
Copy Markdown
Contributor Author

meker12 commented Jun 24, 2025

Two things stand out on cursory review:

  • The links to API Reference 404. Is that expected?
  • The edition: ee metadata key-value pair is no longer supported, which is why pages with that metadata are rendering the fallback yellow badge. You'll need to update that.

I fixed the 404 links. I can't test the API reference correctly until we push to the staging branch -- tried building on gitpages pipeline, but the build fails.

Added edition tags.

@meker12 meker12 requested a review from jeff-matthews June 24, 2025 17:38
Copy link
Copy Markdown
Contributor

@jeff-matthews jeff-matthews left a comment

Choose a reason for hiding this comment

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

LGTM 😄

@github-project-automation github-project-automation bot moved this from 📋 Needs Review to 👍 Approved in Commerce - Pull Requests Jun 24, 2025
@meker12 meker12 merged commit daa9398 into ccdm-early-access Jun 24, 2025
10 of 11 checks passed
@meker12 meker12 deleted the DCAT-2416-refactor-navigation branch June 24, 2025 18:35
@github-project-automation github-project-automation bot moved this from 👍 Approved to 🏁 Done in Commerce - Pull Requests Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

major-update Significant original updates to existing content

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

7 participants