Skip to content

Comments

Update organization.md#360

Merged
d-claassen merged 1 commit intomainfrom
d-claassen/organization-piece-name
May 22, 2025
Merged

Update organization.md#360
d-claassen merged 1 commit intomainfrom
d-claassen/organization-piece-name

Conversation

@d-claassen
Copy link
Contributor

Summary

Reflect that the name property is part of the Organization. It's not placed under @id.

Relevant technical choices

Technical choices that affect more than this issue:

Test instructions

This PR can be acceptance tested by following these steps:
1.
1.
1.

Quality assurance

  • Security - I have thought about any security implications this code might add.
  • Performance - I have checked that this code doesn't impact performance (greatly).
  • Caching - I have analyzed the caching methods that this code touches and have added instructions to deal with those.
  • Tested - I have tested this code to the best of my abilities.
  • Automated tests - I have added unit tests to verify the code works as intended.
  • Testability - I have added unique ids to elements, so they can be located in automated testing.
  • I have altered a filename.
    • I have adjusted the ID property accordingly and updated all internal links.
    • I have added the redirect to the _redirects file in the root of the project.

Reflect that the `name` property is part of the `Organization`. It's not placed under `@id`.
@cloudflare-workers-and-pages
Copy link

Deploying yoast-developer with  Cloudflare Pages  Cloudflare Pages

Latest commit: bd2f5f5
Status: ✅  Deploy successful!
Preview URL: https://b1e82f17.yoast-developer.pages.dev
Branch Preview URL: https://d-claassen-organization-piec.yoast-developer.pages.dev

View logs

@d-claassen
Copy link
Contributor Author

Can confirm proper nesting:

image

@d-claassen d-claassen merged commit 8ba8d7c into main May 22, 2025
1 check passed
@d-claassen d-claassen deleted the d-claassen/organization-piece-name branch May 22, 2025 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant