Skip to content

Conversation

@Sugandhgyl
Copy link
Contributor

Please ensure your pull request adheres to the following guidelines:

  • make sure to link the related issues in this description
  • when merging / squashing, make sure the fixed issue references are visible in the commits, for easy compilation of release notes

Related Issues

Thanks for contributing!

@github-actions
Copy link

This PR will trigger a minor release when merged.

@Sugandhgyl Sugandhgyl changed the title feat: Implement Tag Mappings for Opportunity Creation and Updates feat: Implement Tag Mappings for Opportunities Jan 13, 2026
@Sugandhgyl Sugandhgyl force-pushed the opportunity-tag-mappings branch from 29f2b5b to 4b06acf Compare January 13, 2026 12:51
@Sugandhgyl Sugandhgyl requested review from Kanishkavijay39 and habansal and removed request for Kanishkavijay39 January 13, 2026 12:52
Copy link

@anshulk-public anshulk-public left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@tarunsinghdev tarunsinghdev left a comment

Choose a reason for hiding this comment

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

LGTM.

HEADINGS: 'headings',
HREFLANG: 'hreflang',
INVALID_OR_MISSING_METADATA: 'meta-tags',
PRODUCT_METATAGS: 'product-metatags',
Copy link
Contributor

Choose a reason for hiding this comment

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

jfyi, this is a Adobe Commerce Optimizer only opportunity.
and would have isACO tag.
Do you also need to preserve that tag?

won't show up in ASO UI, fwik.
But, please cross-check.

Adding a constant here is fine, but just letting you know in case you have any changes related to this in other PRs

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.

5 participants