Skip to content

Fix webhook active check and improve admin plugin edit screen#354

Merged
simonhamp merged 1 commit intomainfrom
fix-webhook-active-check
Apr 13, 2026
Merged

Fix webhook active check and improve admin plugin edit screen#354
simonhamp merged 1 commit intomainfrom
fix-webhook-active-check

Conversation

@simonhamp
Copy link
Copy Markdown
Member

Summary

  • Webhook endpoint: Only reject plugins where is_active is false instead of checking approval status — allows all active plugins to receive webhook data regardless of their review status
  • Admin plugin edit: Link composer package name to Packagist for free plugins, hide repo URL for paid third-party plugins, link license to license page for paid plugins (GitHub for free), and add a "Go to User" action on the submitter field
  • Plugin model: Add isActive() method and inactive() factory state

Test plan

  • Webhook rejects inactive plugins with 403
  • Webhook allows unapproved but active plugins
  • Ping events still work for all plugins
  • Free plugin name links to Packagist
  • Paid plugin name shows plain text
  • Paid third-party plugin hides repository URL
  • Paid official plugin still shows repository URL
  • Paid plugin license links to license page
  • Free plugin license links to GitHub LICENSE
  • Go to User action visible on edit plugin page

🤖 Generated with Claude Code

Change webhook endpoint to only reject inactive plugins (is_active=false)
instead of unapproved ones, allowing all active plugins to receive webhook
data regardless of approval status.

Admin plugin edit improvements:
- Link composer package name to Packagist for free plugins
- Hide repository URL for paid third-party plugins
- Link license to license page for paid plugins, GitHub for free
- Add 'Go to User' action on submitter field in Submission Info

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@simonhamp simonhamp marked this pull request as ready for review April 13, 2026 00:08
@simonhamp simonhamp merged commit 2f6a574 into main Apr 13, 2026
2 checks passed
@simonhamp simonhamp deleted the fix-webhook-active-check branch April 13, 2026 00:17
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