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

Set the rich subject #53

Merged
merged 1 commit into from
Nov 4, 2016
Merged

Conversation

nickvergessen
Copy link
Member

Steps

  1. Create an announcement
  2. Run cron job
  3. See that the announcement subject is now clickable in the notification.

@harry-7 do you want to test this? 😄

@hkveeranki
Copy link

@nickvergessen Sure. I checked it out. It seems like this patch is interfering with the notifications app. Check this out once and tell me if I am wrong

Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen nickvergessen force-pushed the notification-rich-object-strings branch from 5f5ebdf to fb570ac Compare November 4, 2016 12:03
@nickvergessen nickvergessen merged commit 3be79a8 into master Nov 4, 2016
@nickvergessen nickvergessen deleted the notification-rich-object-strings branch November 4, 2016 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants