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.

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

Email/Slack Alerts assume all alerts are for posts #1351

Open
dougaxe1 opened this issue Jul 8, 2022 · 0 comments
Open

Email/Slack Alerts assume all alerts are for posts #1351

dougaxe1 opened this issue Jul 8, 2022 · 0 comments

Comments

@dougaxe1
Copy link

dougaxe1 commented Jul 8, 2022

Bug Report

Expected Behavior

Alerts should not assume record object_id is a post and should display alert actions based on the proper connector.

Actual Behavior

Alerts assume $recordarr['object_id'] is always a post and as a result offer a confusing message and "Edit %s" link, even if the object_id represents a user ID (or similar).

Steps to Reproduce the Problem

  1. Create an Email or Slack alert on the User -> Profiles action

  2. Edit a user's profile

  3. View message sent

Sample message

If the user ID matched a post ID, you'll see a message like this:

A Stream Alert was triggered on [Site].

User:
Context: Profiles
Action: Updated
Page: [Some Unrelated Page]

Edit Page
<>

Edit Alert
<[example.com]/wp-admin/edit.php?post_type=wp_stream_alerts#post-123456>

System Information

  • Stream plugin version: 3.9.0
  • WordPress version: 5.9.3
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

No branches or pull requests

1 participant