Skip to content

Conversation

JV0812
Copy link
Collaborator

@JV0812 JV0812 commented Aug 6, 2025

Purpose of this pull request

This pull request is to update the keywords in 10 Azure app docs.

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

Improve the app docs SEO (Azure)

@JV0812 JV0812 requested review from kimsauce, jpipkin1, mafsumo, amee-sumo and a team as code owners August 6, 2025 12:44
@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Aug 6, 2025
@JV0812 JV0812 self-assigned this Aug 6, 2025
@JV0812 JV0812 requested a review from kimsauce August 7, 2025 11:47
@JV0812 JV0812 marked this pull request as draft August 8, 2025 13:45
@JV0812 JV0812 marked this pull request as ready for review August 10, 2025 07:16
@jpipkin1
Copy link
Collaborator

jpipkin1 commented Aug 11, 2025

@JV0812 and @kimsauce, do we need to update the existing front matter keyword sections in all our articles so they follow this pattern:

keywords: [keyword 1, keyword 2, keyword 3]

instead of this pattern?

keywords:
- keyword 1
- keyword 2
- keyword 3

@JV0812
Copy link
Collaborator Author

JV0812 commented Aug 12, 2025

@jpipkin1 Thanks for keeping an eye on these. I am good with updating it, specially in our release notes. But still, I would suggest @kimsauce to take a call on this.

@kimsauce
Copy link
Collaborator

@jpipkin1 , great catch! The Docusaurus link I provided was incorrect. That was frontmatter formatting for one-off pages (/src/pages/); the frontmatter formatting for /docs/ (which we use globally) is here.

@JV0812 , would you pls correct to follow the regular formatting?
Screenshot 2025-08-13 at 2 08 00 AM

Copy link
Collaborator

@kimsauce kimsauce left a comment

Choose a reason for hiding this comment

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

@JV0812 JV0812 requested a review from kimsauce August 18, 2025 13:41
Copy link
Collaborator

@jpipkin1 jpipkin1 left a comment

Choose a reason for hiding this comment

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

Looks good, @JV0812!

@JV0812 JV0812 dismissed kimsauce’s stale review August 26, 2025 14:17

Thanks Kim, I have fixed your comments!

@JV0812 JV0812 enabled auto-merge August 26, 2025 14:17
@JV0812 JV0812 added this pull request to the merge queue Aug 26, 2025
Merged via the queue into main with commit 6c4fed3 Aug 26, 2025
7 checks passed
@JV0812 JV0812 deleted the SEO-optimization-(Pilot) branch August 26, 2025 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed Contributor approved, listed in .clabot file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants