AminoJS / Amino.JS

A powerful JavaScript library for interacting with the Amino API :star2:
MIT License
27 stars 5 forks source link

Author undefined in getChat function #42

Closed moelrobi closed 5 years ago

moelrobi commented 6 years ago

Describe the bug In amino.getChat() there is a bug that the authors name is not defined

Expected behavior author.name is undefinded, due to backend change.

felixfong227 commented 6 years ago

Question @moelrobi I can't even see there is an author.name key inside the return object image

moelrobi commented 6 years ago

helpers/sorter.js:

sendMessageSorter: (uid, element) => {

return {

'threadId': uid,

'messageId': element.messageId,

'msg': element.content,

'type': element.type,

'author': {

'uid': element.author.uid,

'name': element.author.name,

'level': element.author.level,

'role': element.author.role

}

};

},

The Backend now needs

element.author.nickname

Sent from my Galaxy S7 using FastHub

felixfong227 commented 5 years ago

Fixed the issue at https://github.com/AminoJS/Amino.JS/tree/issue42 😄

akatsukilevi commented 5 years ago

I tested the issue fix, works perfectly

felixfong227 commented 5 years ago

@akatsukilevi You mean the branch I pushed or you are referring another branch 😹

akatsukilevi commented 5 years ago

The one you pushed https://github.com/AminoJS/Amino.JS/tree/issue42 (sorry for late reply)

felixfong227 commented 5 years ago

Thanks @akatsukilevi , I’m going to close this issue now, if this problem occurred again(which I think it will🤣) we shall reopen this issue or just simply open an other issue ticket

moelrobi commented 5 years ago

It should not.

Felix Fong notifications@github.com schrieb am Do., 18. Okt. 2018, 19:11:

Thanks @akatsukilevi https://github.com/akatsukilevi , I’m going to close this issue now, if this problem occurred again(which I think it will 🤣) we shall reopen this issue or just simply open an other issue ticket

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AminoJS/Amino.JS/issues/42#issuecomment-431089508, or mute the thread https://github.com/notifications/unsubscribe-auth/AQ0th4Skst8Iyu4Vx4ZYvt21PLUtu_Opks5umLZZgaJpZM4WLKLc .

akatsukilevi commented 5 years ago

You're welcome!

Thanks @akatsukilevi , I’m going to close this issue now, if this problem occurred again(which I think it will) we shall reopen this issue or just simply open an other issue ticket