callmearta / clubhouse-desktop

An unofficial Clubhouse desktop client
Other
214 stars 38 forks source link

Unable to follow #24

Closed eamirgh closed 3 years ago

eamirgh commented 3 years ago

There is no UI to follow people.

miladini commented 3 years ago

I changed the following line https://github.com/callmearta/clubhouse-desktop/blob/150ba375cdb917651c57b993b851aaf501f9fd75/app/views/user.js#L324

into:

                    <button class="btn-primary mt-3" @click="follow" v-if="!isMe && user.user_profile.notification_type !== 2">Follow</button>

and it worked for me.

eamirgh commented 3 years ago

@miladini why don't you create a pull request?

miladini commented 3 years ago

@eamirgh

There you go! https://github.com/callmearta/clubhouse-desktop/pull/25

eamirgh commented 3 years ago

@miladini thank you so much 🍻

callmearta commented 3 years ago

Follow button should be handled by checking user's data which contains followings ids, not notification type. Before it was alright to check it with notification type but not anymore. If you'd like to do a pull request i appreciate it.