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.

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

[GTIN] Follow-up - Full coverage for GTIN logic. #2687

Open
puntope opened this issue Nov 22, 2024 · 0 comments
Open

[GTIN] Follow-up - Full coverage for GTIN logic. #2687

puntope opened this issue Nov 22, 2024 · 0 comments
Assignees
@puntope
Labels
priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. type: technical debt This issue/PR represents/solves the technical debt of the project.

Comments

@puntope
Copy link
Contributor

puntope commented Nov 22, 2024

Related to #2617

Is your feature request related to a problem?

The Unit tests coverage for GTIN logic is not completed 100% missing some aspects as

  • YOAST SEO integration support
  • Variations
  • GTIN MIgration Controller
  • WPCLI Tool
  • UI MIgration Banner

As reference: Find here the CODECOV report to undertdand whats missing in the test coverage:

https://app.codecov.io/gh/woocommerce/google-listings-and-ads/pull/2684?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=woocommerce
@puntope puntope added the type: enhancement The issue is a request for an enhancement. label Nov 22, 2024
@puntope puntope self-assigned this Nov 22, 2024
@puntope puntope added type: technical debt This issue/PR represents/solves the technical debt of the project. priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. and removed type: enhancement The issue is a request for an enhancement. labels Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@puntope puntope

Labels
priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. type: technical debt This issue/PR represents/solves the technical debt of the project.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@puntope