Shopify\Webhooks\Topics is missing 38 topics from the WebhookSubscriptionTopic enum (through 2026-01/2026-07)

未关闭 适合新手
#470 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
78/100
Issue 类型
功能
描述清晰度
描述清楚
活跃度
活跃
技术栈
php
领域
api

调研方向

首先检查 Shopify\Webhooks\Topics,并将其中的常量与列出的 WebhookSubscriptionTopic 枚举项进行比较。添加缺失的 topic,并确认是否应包含额外的 finance 和 returns 项。完成的标准是这些常量可用于 Registry::register() 和 Registry::addHandler(),而不再需要原始字符串。

由索引模型根据 Issue 内容生成。

描述

Issue summary

Shopify\Webhooks\Topics currently exposes 188 constants. The Admin GraphQL
WebhookSubscriptionTopic enum (API version 2026-07) defines 32 topics that have
no corresponding constant in the class.

Missing topics:

  • APP_SCOPES_UPDATE
  • CUSTOMERS_PURCHASING_SUMMARY
  • CUSTOMERS_WHATS_APP_MARKETING_CONSENT_UPDATE
  • CUSTOMER_JOINED_SEGMENT
  • CUSTOMER_LEFT_SEGMENT
  • DELIVERY_PROMISE_SETTINGS_UPDATE
  • FULFILLMENT_HOLDS_ADDED
  • FULFILLMENT_HOLDS_RELEASED
  • FULFILLMENT_ORDERS_HOLD_RELEASED
  • FULFILLMENT_ORDERS_MANUALLY_REPORTED_PROGRESS_STOPPED
  • FULFILLMENT_ORDERS_PROGRESS_REPORTED
  • INVENTORY_SHIPMENTS_ADD_ITEMS
  • INVENTORY_SHIPMENTS_CREATE
  • INVENTORY_SHIPMENTS_DELETE
  • INVENTORY_SHIPMENTS_MARK_IN_TRANSIT
  • INVENTORY_SHIPMENTS_RECEIVE_ITEMS
  • INVENTORY_SHIPMENTS_REMOVE_ITEMS
  • INVENTORY_SHIPMENTS_UPDATE_ITEM_QUANTITIES
  • INVENTORY_SHIPMENTS_UPDATE_TRACKING
  • INVENTORY_TRANSFERS_ADD_ITEMS
  • INVENTORY_TRANSFERS_CANCEL
  • INVENTORY_TRANSFERS_COMPLETE
  • INVENTORY_TRANSFERS_READY_TO_SHIP
  • INVENTORY_TRANSFERS_REMOVE_ITEMS
  • INVENTORY_TRANSFERS_UPDATED
  • INVENTORY_TRANSFERS_UPDATE_ITEM_QUANTITIES
  • LOCALES_DESTROY
  • MARKETS_BACKUP_REGION_UPDATE
  • METAFIELD_DEFINITIONS_CREATE
  • METAFIELD_DEFINITIONS_DELETE
  • METAFIELD_DEFINITIONS_UPDATE
  • ORDERS_LINK_REQUESTED
Expected behavior

Topics lists these topics as constants, so apps can subscribe with
Topics::METAFIELD_DEFINITIONS_CREATE rather than a raw string.

Actual behavior

The constants don't exist, so the raw string has to be passed to
Registry::register() / Registry::addHandler(). This still works — the docs
note the class "may not be completely up to date at all times" — but it loses
the discoverability and typo-safety the class exists to provide.

Source
Notes

The enum additionally contains FINANCE_APP_STAFF_MEMBER_{DELETE,GRANT,REVOKE,UPDATE},
FINANCE_KYC_INFORMATION_UPDATE and RETURNS_PROCESS, which are not listed on the
webhooks topic page. Happy to include or omit those depending on maintainer preference.

主要语言
PHP
星标
473
派生
193
平均合并
5 小时 8 分钟
30 天内合并 PR
1

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

Shopify/shopify-api-php 的其他 Issue

查看 Shopify/shopify-api-php 的全部 Issue

相似的 Issue

更多 PHP Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。