-
Notifications
You must be signed in to change notification settings - Fork 38
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
Rebuild communication and activity feed + assign a comment as task to user #3932
base: main
Are you sure you want to change the base?
Conversation
@theskumar @frjo I presented this today to the program team and everyone really liked the idea of combining the comments & activities tab - feedback I got was that folks sometimes found it inconvenient to flip back & forth between tabs trying to figure out when different events happened and correlating that to the comments. |
also received some enthusiasm around avatars! |
Glad to hear! I'm resuming work on it and keep you posted. |
23ef830
to
0184369
Compare
eec6ead
to
e099ad2
Compare
e099ad2
to
92d1605
Compare
fd623a8
to
a5ba323
Compare
This should too good to review for UX/UI Pending items.
|
3c98f74
to
a67fcf7
Compare
a67fcf7
to
ac884fa
Compare
2992946
to
bfb7af9
Compare
bfb7af9
to
6200d2b
Compare
6200d2b
to
7b766c4
Compare
e2299b1
to
431aae5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works perfectly in my testing.
Fixes #3048
Fixes #4029
Async comment add (won't be done in this PR)Work in progress screenshots
Test Steps