Skip to content

Link new plugin notification to specific plugin page#314

Merged
simonhamp merged 2 commits into
mainfrom
plugin-notification-links
Mar 29, 2026
Merged

Link new plugin notification to specific plugin page#314
simonhamp merged 2 commits into
mainfrom
plugin-notification-links

Conversation

@simonhamp

Copy link
Copy Markdown
Member

Summary

  • Email notification "View Plugin" action now links to the specific plugin page (/plugins/{vendor}/{package}) instead of the generic /plugins directory
  • Database notification data now includes action_url and action_label fields
  • Notifications Blade template renders a clickable action button when action_url is present (generic pattern, works for any notification)

Test plan

  • New test: test_mail_action_links_to_plugin_page asserts correct URL in email
  • Extended test_database_notification_contains_plugin_data to assert action_url and action_label
  • All 8 notification tests pass

🤖 Generated with Claude Code

simonhamp and others added 2 commits March 29, 2026 22:02
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ttons on mobile

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@simonhamp simonhamp marked this pull request as ready for review March 29, 2026 21:16
@simonhamp simonhamp merged commit 0dd5379 into main Mar 29, 2026
2 checks passed
@simonhamp simonhamp deleted the plugin-notification-links branch March 29, 2026 21:16
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