-
Notifications
You must be signed in to change notification settings - Fork 4
RO with 12 existing ROs #7
Copy link
Copy link
Open
Labels
bugSomething isn't workingSomething isn't workingenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomersminorDoes not cause significant issuesDoes not cause significant issues
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomersminorDoes not cause significant issuesDoes not cause significant issues
Attempting to RO when the region already has 12 ROs may result in a failure to successfully appoint oneself.
This can be circumvented by changing line 286 in content.js from
if(encounteredSelf) {to
if(encounteredSelf || other_ros.length == 12) {This allows YAFFeather to dismiss an RO if it has not found itself, but there are 12 ROs, meaning one must be dismissed before appointing.
This allows YAFFeather to catch failed RO attempts due to no available RO slots.