Skip to content
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

IM notification sounds still played when away #24

Closed
jaymzh opened this Issue Jul 12, 2015 · 6 comments

Comments

Projects
None yet
2 participants
@jaymzh
Copy link

jaymzh commented Jul 12, 2015

My pidgin is configured to only make sounds on received IMs when "available" - but the FB plugin doesn't seem to obey this and I get sounds when away.

@jgeboski

This comment has been minimized.

Copy link
Collaborator

jgeboski commented Jul 12, 2015

So, the plugin does not define any sort of "away" state, which is likely why this happens. This is due to the fact that Facebook only supports "Available" and "Offline." I am going to have to bother some of the other Pidgin developers to hopefully figure out the best course of action.

@jaymzh

This comment has been minimized.

Copy link
Author

jaymzh commented Jul 12, 2015

I figured as much. I'd just make a dummy away state that doesn't change the state from Facebook's perspective (still "available"), but allows libpurple to know you're away.

BTW, this software is stellar - thanks so much for saving my ability to communicate with FB friends via my preferred chat platform. So happy.

@jgeboski

This comment has been minimized.

Copy link
Collaborator

jgeboski commented Jul 20, 2015

So, I talked to two of the pidgin developers, and they were pretty neutral on the matter of this being implemented. It is good for the people, such as you, and bad for people who expect to be away. But it seems there might be some sort of idle presence state, which could be used for the "Away" state.

I think the general consensus is a better way to handle presence states in purple3. But for now, I am going to go ahead implement this at some point in the near future.

@jaymzh

This comment has been minimized.

Copy link
Author

jaymzh commented Jul 20, 2015

That's awesome! Thanks.

@jgeboski

This comment has been minimized.

Copy link
Collaborator

jgeboski commented Jul 25, 2015

I have implemented this in 051e2725dec4. It will appear in the next update.

@jgeboski jgeboski closed this Jul 25, 2015

@jgeboski jgeboski added the resolved label Jul 25, 2015

@jaymzh

This comment has been minimized.

Copy link
Author

jaymzh commented Jul 25, 2015

You are awesome, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.