search

Hirenachhaada / Mern-Chat-Application

1 stars 11 forks source link

Update group admin when current admin leaves #20

Closed IrfanSalim closed 8 months ago

IrfanSalim commented 9 months ago

Close #5

Added functionality to select the next group admin when the current group admin is leaving the group

image

Note: while starting the application, I was getting some cors error hence had to update some other things

Hirenachhaada commented 9 months ago

Nice work. But one more thing is required. The user can leave the group by clicking on the cross icon against their name. At that moment it's not asking for a change admin option

IrfanSalim commented 9 months ago

@Hirenachhaada I think the easier option to tackle that issue would be to not show the current users name in the other users sections, what do you think?

Hirenachhaada commented 9 months ago

That will also work. Ok just implement it and create a pull request

IrfanSalim commented 9 months ago

Done @Hirenachhaada

IrfanSalim commented 8 months ago

Hey, @Hirenachhaada

Is there an issue?

Hirenachhaada commented 8 months ago

Nice work