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

Seen/Unseen feature #242

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

snyaggarwal
Copy link

Few days back I raised a feature request #240
This pull-request implements that feature.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.7%) to 92.802% when pulling 827fbbd on snyaggarwal:master into 8ba5e32 on django-notifications:master.

@morenoh149
Copy link

morenoh149 commented Jun 11, 2019

seems like the travis build timed out. Has anyone reviewed this?

Would be great to have this feature in Django-notifications because this is exactly lacking in django-activity-stream justquick/django-activity-stream#341

edit: I didn't realize Django-notifications has read/unread this is enough for my requirements. Seen is something else?

@AlvaroLQueiroz
Copy link
Contributor

I'm not sure if it's a useful functionality for the purpose of this lib. Can you provide some examples where we can use this instead of the read functionality?

@koleror
Copy link

koleror commented Oct 11, 2023

Hey there!
Any news about this feature? Would really love to see it merged.
Unless there's a working workaround? Like a way to use a custom model where I could add the missing field?

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.

None yet

5 participants