Skip to content
Snippets Groups Projects
Unverified Commit 4df6bdde authored by graywolf336's avatar graywolf336
Browse files

Add a warning about a new user type possibly being added

parent ca69abc5
No related branches found
No related tags found
No related merge requests found
...@@ -23,8 +23,8 @@ export class AppUsersConverter { ...@@ -23,8 +23,8 @@ export class AppUsersConverter {
} }
const type = this._convertUserTypeToEnum(user.type); const type = this._convertUserTypeToEnum(user.type);
const status = this._convertStatusConnectionToEnum(user.status); const status = this._convertStatusConnectionToEnum(user.username, user._id, user.status);
const statusConnection = this._convertStatusConnectionToEnum(user.statusConnection); const statusConnection = this._convertStatusConnectionToEnum(user.username, user._id, user.statusConnection);
return { return {
id: user._id, id: user._id,
...@@ -50,11 +50,12 @@ export class AppUsersConverter { ...@@ -50,11 +50,12 @@ export class AppUsersConverter {
case 'bot': case 'bot':
return UserType.BOT; return UserType.BOT;
default: default:
console.warn(`A new user type has been added that the Apps don't know about? "${ type }"`);
return type.toUpperCase(); return type.toUpperCase();
} }
} }
_convertStatusConnectionToEnum(status) { _convertStatusConnectionToEnum(username, userId, status) {
switch (status) { switch (status) {
case 'offline': case 'offline':
return UserStatusConnection.OFFLINE; return UserStatusConnection.OFFLINE;
...@@ -65,7 +66,7 @@ export class AppUsersConverter { ...@@ -65,7 +66,7 @@ export class AppUsersConverter {
case 'busy': case 'busy':
return UserStatusConnection.BUSY; return UserStatusConnection.BUSY;
default: default:
console.warn(`A user does not have a valid status (offline, online, away, or busy). It is currently: "${ status }"`); console.warn(`The user ${ username } (${ userId }) does not have a valid status (offline, online, away, or busy). It is currently: "${ status }"`);
return status === '' ? UserStatusConnection.OFFLINE : status.toUpperCase(); return status === '' ? UserStatusConnection.OFFLINE : status.toUpperCase();
} }
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment