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 NGINX Instance Manager product/nginx-plus NGINX Plus (and NGINX Open Source in docs.ngninx.com) product/nginx-one NGINX One Console product/nginxaas NGINX as a Service for Azure product/agent NGINX Agent product/amplify NGINX Amplify (EOS product) product/controller NGINX Controller (EOS product) product/mesh NGINX Service Mesh (EOS product) product/nap-dos NGINX App Protect DOS product/nap-waf NGINX App Protect WAF product/nms 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 <3287976+mjang@users.noreply.github.com>

---------

Co-authored-by: Mike Jang <3287976+mjang@users.noreply.github.com>
@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 NGINX Agent product/amplify NGINX Amplify (EOS product) product/controller NGINX Controller (EOS product) product/mesh NGINX Service Mesh (EOS product) product/nap-dos NGINX App Protect DOS product/nap-waf NGINX App Protect WAF product/nginx-one NGINX One Console product/nginx-plus NGINX Plus (and NGINX Open Source in docs.ngninx.com) product/nginxaas NGINX as a Service for Azure product/nim NGINX Instance Manager product/nms 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