Skip to content

Add plugin submission checks and require support channel#305

Merged
simonhamp merged 3 commits intomainfrom
plugin-submission-checks
Mar 28, 2026
Merged

Add plugin submission checks and require support channel#305
simonhamp merged 3 commits intomainfrom
plugin-submission-checks

Conversation

@simonhamp
Copy link
Copy Markdown
Member

Summary

  • License file & release version checks: ReviewPluginRepository now verifies that the repo contains a license file (LICENSE, LICENSE.md, or LICENSE.txt) and has a GitHub release or tag. These are required checks — plugins cannot be approved until both pass.
  • Approval gating: The Filament admin approve action is disabled when required checks fail, with a message explaining which checks are missing.
  • Customer-facing review status: The plugin dashboard splits review checks into "Required for approval" (license, release version) and "Additional checks" (iOS, Android, JS, SDK) with distinct visual indicators.
  • Support channel required: The support channel field is now required on plugin submission and must be a valid email address or URL. The automated "support email in README" check has been removed.
  • Editable support channel: Users can update their support channel from the plugin edit page on their dashboard.

Test plan

  • 18 tests pass in ReviewPluginRepositoryTest (license file detection, release version detection, full integration)
  • 2 tests pass in CustomerPluginReviewChecksTest (submission runs checks, notification includes failing checks)
  • 3 tests pass in PluginReviewChecksIncompleteTest (email rendering, all-failing, subject line)
  • 9 tests pass in PluginSubmissionNotesTest (notes, support channel email/URL, required validation, invalid format validation)
  • Pint passes with no issues

🤖 Generated with Claude Code

simonhamp and others added 3 commits March 28, 2026 12:42
…on submission

- Add automated checks for license file (LICENSE/LICENSE.md/LICENSE.txt) and
  release version (GitHub releases/tags) in ReviewPluginRepository
- Block plugin approval in Filament admin until required checks pass
- Split customer-facing review checks into required and additional sections
- Remove automated support email extraction from README
- Make support channel required on plugin submission with email/URL validation
- Allow users to edit support channel from their plugin dashboard
- Update all related tests and notifications

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ntroller

- Include webhook_installed in required approval checks (Plugin model,
  Filament admin, notifications, customer dashboard)
- Move webhook setup instructions into the review checks section
- Update best-practices docs: replace support email with support channel,
  add license/release/webhook requirements
- Delete dead code: CustomerPluginController, its form requests, and old
  customer/plugins Blade views (replaced by Livewire components)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
# Conflicts:
#	resources/views/customer/plugins/create.blade.php
#	resources/views/customer/plugins/index.blade.php
@simonhamp simonhamp marked this pull request as ready for review March 28, 2026 13:37
@simonhamp simonhamp merged commit 7dcada3 into main Mar 28, 2026
2 checks passed
@simonhamp simonhamp deleted the plugin-submission-checks branch March 28, 2026 13:43
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