Comments (8)
Maybe add a MumbleClient.usersOnline() and MumbleClient.usersOffline() method as well as a property "online" to the user objects instead of changing the behaviour of this method?
from node-mumble.
Maybe add a MumbleClient.usersOnline() and MumbleClient.usersOffline() method as well as a property "online" to the user objects instead of changing the behaviour of this method?
Yes, this seems like the best solution instead of making breaking changes.
from node-mumble.
In terms of keeping the API clean and logical, I wouldn't consider this breaking change. The session IDs, etc. are wrong anyway for disconnected users. If the application needs to keep track of all the possible users, they should persist them somewhere.
.. and how did I miss the original issue posted October 18th? This completely went past me. Sorry for that!
from node-mumble.
(I'd be okay with usersOnline() and usersOffline() if users() returned -all- the registered users from the server. Even the ones who haven't logged on at all.)
Returning users the client has seen is just iffy.
(On the other hand: While fixing this, I think we should also include a method to retrieve all registered users in case the bot account has necessary permissions for that.)
from node-mumble.
While fixing this, I think we should also include a method to retrieve all registered users in case the bot account has necessary permissions for that.
Does a user require a special permission to view all registered users? I'm not seeing any ACL for that:
from node-mumble.
Not sure about that. Could be everyone can see them.
from node-mumble.
Hm, I went on a random server and I couldn't access registered users.
from node-mumble.
Could be the 'register user' permission affects it.
from node-mumble.
Related Issues (20)
- Trouble authenticating user HOT 2
- Events in the client HOT 7
- Register user HOT 9
- Deaf state not updated on user
- user.inputStream() fails with TypeError: cannot set whisperId of undefined HOT 2
- Not receiving private message event? HOT 4
- Is this project dead? HOT 3
- Update dependencies HOT 3
- BIZZARE WEIRD ERRORS UPON INSTALLATION HOT 4
- Play audio from channel (server) HOT 4
- Not recording from subgroup. HOT 1
- npm install not working anymore HOT 3
- Positional Audio HOT 1
- Install error 886 verbose stack Error: [email protected] install: `node-gyp rebuild` HOT 2
- v8 warning for native addon on node v6+ HOT 4
- How do I set the comment for the connected user? HOT 5
- Fatal error in api.cc HOT 6
- Can't run sendMessage function from outside mumble.connect HOT 2
- Windows 10 disconnects after a few seconds HOT 1
- Is there an example how to play/stream audio to Web Audio API ? HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from node-mumble.