rootabeta / YAFFeather

Yet Another Fork of Feather
MIT License
0 stars 2 forks source link

RO with 12 existing ROs #7

Open rootabeta opened 8 months ago

rootabeta commented 8 months ago

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.

rootabeta commented 7 months ago

Merged to beta in https://github.com/rootabeta/YAFFeather/commit/8b89379bc39614eaeff99e8d4ca716c772104d10, awaiting field tests