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.

Sign up for GitHub

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

Jump to bottom

Dismiss annoucements #209

Open
potomak opened this issue Dec 2, 2016 · 1 comment
Open

Dismiss annoucements #209

potomak opened this issue Dec 2, 2016 · 1 comment
Labels
feature hacktoberfest help wanted

Comments

@potomak
Copy link
Member

potomak commented Dec 2, 2016

Announcements should be visible only as soon as a user dismiss them.
@potomak potomak added this to the 1.0 milestone Dec 2, 2016
@potomak
Copy link
Member Author

potomak commented Jan 22, 2017

Instead of storing a new value in the user model a simple flag in the session could be sufficient.

An alternative approach would be to create a new Announcement model that could be used to store announcements and associations between them and users to store dismissals for logged in users. This way the application wouldn't even need to be restarted to show a new announcement.

@potomak potomak modified the milestone: Version 1.0 Jan 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature hacktoberfest help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@potomak