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

feat: markasread in dropdown #3314

Merged
merged 10 commits into from
Nov 17, 2022
Merged

Conversation

VaibhavUpreti
Copy link
Member

@VaibhavUpreti VaibhavUpreti commented Oct 11, 2022

Fixes #3313

Describe the changes you have made in this PR -

  1. Added mark as read link in notification drop down

Screenshots of the changes (If any) -

BEFORE FIX
beforefix

AFTER FIX
afterfix

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

@coveralls
Copy link

coveralls commented Oct 11, 2022

Coverage Status

Coverage decreased (-0.06%) to 81.977% when pulling 72aff70 on VaibhavUpreti:markasread into 6803ef1 on CircuitVerse:master.

db/schema.rb Outdated Show resolved Hide resolved
@VaibhavUpreti
Copy link
Member Author

VaibhavUpreti commented Oct 12, 2022

@tachyons Removed the unwanted commits , please have a look. Thanks 👍

@VaibhavUpreti
Copy link
Member Author

Unread notifications-
Screenshot 2022-11-01 at 11 13 24 PM
When no unread notifications-

Screenshot 2022-11-01 at 11 14 01 PM

app/assets/stylesheets/layout.scss Outdated Show resolved Hide resolved
app/assets/stylesheets/layout.scss Outdated Show resolved Hide resolved
app/assets/stylesheets/layout.scss Outdated Show resolved Hide resolved
app/assets/stylesheets/layout.scss Outdated Show resolved Hide resolved
@VaibhavUpreti
Copy link
Member Author

Fixes issue #3313 .. @tachyons @vedant-jain03 kindly review this.

@vedant-jain03 vedant-jain03 added the notifications the issues and PR for this will under notification context label Nov 3, 2022
@VaibhavUpreti
Copy link
Member Author

@tachyons @vedant-jain03 is this good for mergre?

@vedant-jain03 vedant-jain03 moved this from Ready for Review to Approved / Ready to merge in Main Platform Improvement Nov 16, 2022
config/routes.rb Outdated Show resolved Hide resolved
@codeclimate
Copy link

codeclimate bot commented Nov 17, 2022

Code Climate has analyzed commit 72aff70 and detected 0 issues on this pull request.

View more on Code Climate.

@VaibhavUpreti
Copy link
Member Author

@vedant-jain03 @tachyons updated "get" to "patch" ... also had to change one spec 👍 ... all tests passing

@tachyons tachyons merged commit d3618c1 into CircuitVerse:master Nov 17, 2022
@vedant-jain03 vedant-jain03 moved this from Approved / Ready to merge to Completed(Merged) in Main Platform Improvement Nov 17, 2022
@VaibhavUpreti VaibhavUpreti deleted the markasread branch November 17, 2022 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notifications the issues and PR for this will under notification context
Projects
Main Platform Improvement
Completed(Merged)
Development

Successfully merging this pull request may close these issues.

feat: Include mark as read button in the drop down menu of notifications
4 participants