Archipelago icon indicating copy to clipboard operation
Archipelago copied to clipboard

Add a means to allow a client to opt into seeing ALL hints.

Open CaitSith2 opened this issue 2 years ago • 6 comments

This works by means of the tag "ShowAllHints". The server notify_hints looks for this tag, and will only send all hints even those that don't concern the slot in any way to clients with the tag.

My main motivation for implementing this is so that AginahBot can specify the "ShowAllHints" tag, since only one player is allowed to connect aginahbot to the server anyways.

CaitSith2 avatar Jun 22 '22 11:06 CaitSith2

Seems to me this could be added to the spectator slots,I dont know if need a new tag for this

Jarno458 avatar Jun 22 '22 15:06 Jarno458

I'm very wary to move forward with a change like this while possible changes to hinting as a whole are being discussed. I think we should put this on a short hold until we figure out which direction we want hinting to go.

Ijwu avatar Jun 22 '22 16:06 Ijwu

Will this send the hints every time someone uses !hint or only when the hint is created? Because I'd prefer to only get pinged by the bot once per hint

Alchav avatar Jun 22 '22 16:06 Alchav

Will this send the hints every time someone uses !hint or only when the hint is created? Because I'd prefer to only get pinged by the bot once per hint

Sounds like a bot problem and not a hinting API problem.

Ijwu avatar Jun 28 '22 00:06 Ijwu

WRONG BUTTON SOZ.

Ijwu avatar Jun 28 '22 00:06 Ijwu

so i think this is good for a spectator slot. but if you prefer to keep it a tag, than plz also include an update to the docs to specify and explain the tag

Jarno458 avatar Jul 18 '22 15:07 Jarno458

Clients can now view hints for all players using the data storage Get hints_team_slot, and a client can also stay up to date on all new hints using the data storage SetNotify on the same key(s)

Jarno458 avatar Jan 02 '23 16:01 Jarno458

@CaitSith2 is this still needed? As Jarno pointed out, the data is now available at least.

Berserker66 avatar Jan 30 '23 04:01 Berserker66

I'm gonna make the executive decision and close this PR since there's been a lot of changes to the API since without new updates. Can always re-open it later.

ThePhar avatar Apr 11 '23 02:04 ThePhar