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

User preferences, notifications and contributors #57

Merged
merged 41 commits into from
Jan 27, 2024
Merged

Conversation

madhums
Copy link
Member

@madhums madhums commented Jun 18, 2023

closes #44 #39 #94

Adds user preferences model with the following notification preferences:

Discussion

  • New discussion on a story I created

Post

  • New post on a discussion I created
  • Anyone posts in a discussion I participate in

Story

  • New story was added

--

Make sure to run once deployed

rake db:migrate
rake preferences:create_missing_preferences

Todo:

  • add user preferences for all the above (models, tests)
  • add user preferences UI
  • add notifications
    • for when new stories are created
    • when someone creates a new discussion on my story
    • when someone creates a new post on a discussion I created
    • when someone posts on a discussion I am participating in
  • add tests for the above notifications
  • add notifications web UI
    • add notifications read/unread
    • use turbo stream to deliver notifications on the fly
    • add visual indicator in the header about the new notifications
    • display count perhaps?
  • add user preferences to navigation
  • add notifications to navigation
  • add email footer with a message saying the user can change their preferences in case they don't wish to receive emails
  • add test to make sure preferences are created after confirmation or invitation acceptation

@madhums madhums force-pushed the feature/notifications branch 2 times, most recently from e75ba52 to 9108c5a Compare June 24, 2023 12:31
@madhums madhums force-pushed the feature/notifications branch 5 times, most recently from f852a49 to 6129695 Compare September 1, 2023 09:44
@madhums madhums force-pushed the feature/notifications branch 7 times, most recently from 3a98388 to f611b68 Compare January 6, 2024 12:44
@madhums madhums marked this pull request as ready for review January 7, 2024 17:09
@madhums madhums changed the title User preferences and notifications User preferences, notifications and contributors Jan 8, 2024
@madhums madhums merged commit 62574af into main Jan 27, 2024
2 checks passed
@madhums madhums deleted the feature/notifications branch January 27, 2024 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

add notifications
1 participant