setRole.ts Error

[louuca]

What needs to be fixed:

Line 29
https://github.com/tovyblox/tovy/blob/main/pages/api/workspace/%5Bid%5D/settings/activity/setRole.ts

Role list from video shown, appears to be returning IDs from Group Role names. Apparently it's possible to have duplicate role IDs for different roles, which I find hard to believe, but the error says otherwise.

Error:

Video:
https://user-images.githubusercontent.com/109313802/205474701-42f7c17d-8942-4264-8856-19d550e839a8.mov

[louuca]

https://cdn.discordapp.com/attachments/1046521281081770136/1048820416090755213/Screen_Recording_2022-12-03_at_8.33.09_PM.mov

Video if GitHub link not working.

[marinofranz]

Is this still happening to you?

[marinofranz]

Closing due to inactivity, feel free to re-open.

[louuca]

Closing due to inactivity, feel free to re-open.

This is still a reoccurring issue. Perhaps if this was replicated rather than passing it on, would've been a better solution?

[louuca]

As far as I understand, the current code parses an integer which means that it fetches the group rank's ID.

This should be changed into fetching the rank name instead as it's apparently possible to have two ranks with the same ID.

[louuca]

With these two having the same rank number/id, it causes conflict and Tovy may incorrectly choose the rank as shown in the video in the first post.

[marinofranz]

Closing due to inactivity, feel free to re-open.

This is still a reoccurring issue. Perhaps if this was replicated rather than passing it on, would've been a better solution?

I feel that the closure of this issue was justified, stale issues should not be kept open. Thanks for indicating that this still happens to you, though, we'll look into it.

[marinofranz]

This should be changed into fetching the rank name instead as it's apparently possible to have two ranks with the same ID.

Rank names can also be the same, so there is really no way around this. We can most likely include a notice somewhere to make sure that users do not have same IDs for their ranks. But, I will leave this to the developer responsible for this section.

[7angr]

This should be changed into fetching the rank name instead as it's apparently possible to have two ranks with the same ID.

Rank names can also be the same, so there is really no way around this. We can most likely include a notice somewhere to make sure that users do not have same IDs for their ranks. But, I will leave this to the developer responsible for this section.

GET https://groups.roblox.com/v1/groups/groupId/roles provides a unique identifier for each role.