Skip to content
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.

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

DEVDOCS-5825: [new] Create store-metafields #480

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

bc-traciporter
Copy link
Collaborator

@bc-traciporter bc-traciporter commented Aug 27, 2024

DEVDOCS-5825

What changed?

Added GraphQL and REST store metafields

Release notes draft

Anything else?

ping {names}

@bc-traciporter bc-traciporter marked this pull request as ready for review August 28, 2024 20:57
kskomro
kskomro previously approved these changes Sep 11, 2024
Copy link
Contributor

@kskomro kskomro left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -0,0 +1,134 @@
# Store metafields

Store metafields allow you to attach a metafield to the store itself. They are suitable for any global information against the store that is not channel specific. Channel metafields are appropriate for any storefront-specific data you need to store in metafields. There are two ways to create a store metafield in Admin GraphQL. You can remove the resource ID or use the store’s Global ID. Examples of each type are below:
Copy link
Contributor

Choose a reason for hiding this comment

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

@zvuki do you think we should recommend one way of doing this over another?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@zvuki what is the preferred way?

Copy link
Contributor

@bookernath bookernath left a comment

Choose a reason for hiding this comment

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

LGTM. I have not tested the requests myself from these docs, but as long as someone has, this is good to go.

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.

3 participants