dirigeants / klasa

A class remix of the Komada Bot Framework
https://klasa.js.org
MIT License
201 stars 87 forks source link

Settings#update() returns the wrong data #1114

Open bdistin opened 4 years ago

bdistin commented 4 years ago

Describe the issue

Settings#update() is returning a string for the updated.next value instead of a role when doing something like conf set adminRole someRoleID. This makes the serializer.stringify method return undefined because string.name is not defined.

Expected and actual behavior

updated.next should be a Role on a Role setting such that serializer.strinigfy returns the role name instead of undefined.

Further details