Skip to content

Removed unnecessary metadata keys and standardized the remaining ones #287

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 14 commits into from
Mar 19, 2025
Merged

Removed unnecessary metadata keys and standardized the remaining ones #287

merged 14 commits into from
Mar 19, 2025

Conversation

travisamartin
Copy link
Contributor

@travisamartin travisamartin commented Mar 13, 2025 •
edited
Loading

Proposed changes

This PR:

  • Removes unnecessary metadata keys
  • Replalces categories and doctypes with type

Removed keys

    "_build",
    "aliases",
    "categories",
    "display_breadcrumb",
    "doctypes",
    "linkTitle",
    "menu",
    "catalog",
    "catalogType",
    "journeys",
    "tags",
    "authors",
    "date",
    "versions"

@github-actions github-actions bot added documentation Improvements or additions to documentation product/nim Issues related to NGINX Instance Manager product/nginx-plus Issues related to NGINX Plus (and NGINX Open Source in docs.nginx.com) product/nginx-one Issues related to NGINX One Console product/nginxaas Issues related to NGINX as a Service for Azure product/agent Issues related to NGINX Agent product/amplify Issues related to NGINX Amplify (EOS product) product/controller Issues related to NGINX Controller (EOS product) product/mesh Issues related to NGINX Service Mesh (EOS product) product/nap-dos Issues related to NGINX App Protect DOS product/nap-waf Issues related to NGINX App Protect WAF product/nms Issues related to NGINX Management Suite (EOS product) labels Mar 13, 2025
@github-actions GitHub Actions
Copy link

✅ Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/docs/287/

JTorreG
JTorreG reviewed Mar 13, 2025
View reviewed changes
JTorreG
JTorreG reviewed Mar 13, 2025
View reviewed changes
JTorreG
JTorreG reviewed Mar 13, 2025
View reviewed changes
JTorreG
JTorreG reviewed Mar 13, 2025
View reviewed changes
JTorreG
JTorreG reviewed Mar 13, 2025
View reviewed changes
JTorreG
JTorreG reviewed Mar 13, 2025
View reviewed changes
JTorreG
JTorreG reviewed Mar 13, 2025
View reviewed changes
JTorreG
JTorreG reviewed Mar 13, 2025
View reviewed changes
@travisamartin travisamartin marked this pull request as ready for review March 13, 2025 19:53
@travisamartin travisamartin requested review from a team as code owners March 13, 2025 19:53
@lamATnginx @mjang @travisamartin
Update call-out shortcode usage in Contributing (#286)
93e5c45 
* chore: Update call-out shortcode usage in Contributing

* Update with suggestion

Co-authored-by: Mike Jang <[email protected]>

---------

Co-authored-by: Mike Jang <[email protected]>
@github-actions github-actions bot added the process documentation Documentation related to how the repository or documention itself is managed. label Mar 13, 2025
ADubhlaoich
ADubhlaoich approved these changes Mar 14, 2025
View reviewed changes
Copy link
Contributor

@ADubhlaoich ADubhlaoich left a comment

Choose a reason for hiding this comment

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

LGTM!

Script might need to be re-ran for NIC once it's part of the documentation repository, but I've also been proactively fixing these as I've gone along, so there may not be that many.

@travisamartin travisamartin merged commit 3229349 into main Mar 19, 2025
9 checks passed
@travisamartin travisamartin deleted the clean-up-metadata branch March 19, 2025 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ADubhlaoich ADubhlaoich ADubhlaoich approved these changes

@JTorreG JTorreG JTorreG approved these changes

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation process documentation Documentation related to how the repository or documention itself is managed. product/agent Issues related to NGINX Agent product/amplify Issues related to NGINX Amplify (EOS product) product/controller Issues related to NGINX Controller (EOS product) product/mesh Issues related to NGINX Service Mesh (EOS product) product/nap-dos Issues related to NGINX App Protect DOS product/nap-waf Issues related to NGINX App Protect WAF product/nginx-one Issues related to NGINX One Console product/nginx-plus Issues related to NGINX Plus (and NGINX Open Source in docs.nginx.com) product/nginxaas Issues related to NGINX as a Service for Azure product/nim Issues related to NGINX Instance Manager product/nms Issues related to NGINX Management Suite (EOS product)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@travisamartin @ADubhlaoich @JTorreG @lamATnginx