Skip to content
This repository was archived by the owner on Dec 29, 2022. It is now read-only.

Let setUserId allow number #16

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Let setUserId allow number #16

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
222e7e3
fix typo in setResponseTime
NoobTW Jan 21, 2018
7f4fc10
setUserId should allow numbers
NoobTW Jan 21, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion lib/MessageFactory.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ class MessageFactory {
* @chainable
*/
setUserId (userId) {
this.user_id = userId;
this.user_id = userId.toString();
return this;
}
/**
Expand Down
2 changes: 1 addition & 1 deletion lib/MessageSet.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ class MessageSet extends MessageFactory {
* @chainable
*/
setUserId (userId) {
this.user_id = userId;
this.user_id = userId.toString();
return this;
}
/**
Expand Down
4 changes: 2 additions & 2 deletions lib/MessageSink.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -334,12 +334,12 @@ class MessageSink {
* @chainable
*/
setUserId (userId) {
this.user_id = userId;
this.user_id = userId.toString();
return this;
}
/**
* Sets the instance response_time field to the given number
* @function setUserId
* @function setResponseTime
* @param {Number} responseTime - the response time of the agent to the user
* message.
* @chainable
Expand Down