Feat: "user followed you" notification #505
Conversation
steven-fox
changed the title
There was a problem hiding this comment.
Looks good!
I gotta think though, it would be better for the sake of understanding the code, to extract a notification card of the common elements used & then create 1 blade components per notification & use a match in the notifications/index.blade.php. It would help remove all of those conditionals & provide better readability.
Maybe in a follow up PR...
Couldn’t agree more! #506 😉😁 |
Closes #503
Changes are a little hacky due to the current way we handle rendering/redirecting notifications.
pinkary.followed.notification.mp4