I really don't like this using the same notification item. I think it's really confusing. We should make a separate one for membership_request_approved.