search

tokenly / tokenly-cms

GNU General Public License v2.0
42 stars 12 forks source link

full post content shows up in notification dropdown #23

Closed cryptonaut420 closed 9 years ago

cryptonaut420 commented 9 years ago

@deweller this is in relation to recently merged pull request #13

looks good with the messages being included in the emails, however this also had the side effect of including the full message content in the dropdown notifications and on the notifications page, which is a bit awkward if their message or post is more than a few sentences long. We will need to figure out some way to specify what content to only show in the email notice as opposed to the on-site account notification

deweller commented 9 years ago

I did attempt to address this in #13.

Here is where the the notification code loads a different notification view for the dropdowns/notification page:

https://github.com/cryptonaut420/tokenly/blob/master/www/slick/App/Meta/Model.php#L269-L281

This is working on my local install. Can you think of any reason why the server might not see this file?

https://github.com/cryptonaut420/tokenly/blob/master/www/themes/views/notifications/forumTopicMention.php

cryptonaut420 commented 9 years ago

This is actually a non-issue, it works fine lol. Turns out I forgot to include the updated App/Meta/Model in the private LTB repo. Updated and uploaded to site, working fine now! :)