Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Announce player revival #82

Merged
merged 2 commits into from
Dec 31, 2024
Merged

Announce player revival #82

merged 2 commits into from
Dec 31, 2024

Conversation

SwissalpS
Copy link
Contributor

@SwissalpS SwissalpS commented Dec 14, 2024

revert some of the eliminations in e99f8e5

@SwissalpS SwissalpS added the enhancement New feature or request label Dec 14, 2024
@SwissalpS SwissalpS self-assigned this Dec 14, 2024
@BuckarooBanzay
Copy link
Contributor

looks good, does this work with the beerchat mod? (haven't checked the code there yet, sorry)

@SwissalpS
Copy link
Contributor Author

IDK, I abborted testing locally as the message didn't come through even without [beerchat] or these changes active.

@SwissalpS
Copy link
Contributor Author

How ironic, I was for the removal back then and now I'm the one adding it back in :D

@SwissalpS SwissalpS merged commit 391e566 into master Dec 31, 2024
2 checks passed
@SwissalpS SwissalpS deleted the announcePlayerRevival branch December 31, 2024 08:53
@SwissalpS
Copy link
Contributor Author

Seems to work for in-game chat but on IRC the messages persist.
Also the messages issued with the chatcommands work in-game but not on IRC.
Is it beerchat or is it another bridge that posts on IRC?

@SwissalpS
Copy link
Contributor Author

SwissalpS commented Jan 2, 2025

found it:
https://github.com/mt-mods/beerchat/blob/4b146ccc02eb3d7ea8a100722163d45312b14dbd/web/tx.lua#L42-L63

I see two ways of going about this:

  1. expose the functions in [beerchat] so we can override them in [pandorabox_custom]
  2. add this as a feature to [beerchat]

Any other ideas? And which path is cleanest?

Edit: if 2. then probably best to change the name of the priv to no_announce_join_leave
(or similar) as there is another announce feature in [beerchat].

@SwissalpS
Copy link
Contributor Author

  1. override beerchat.on_channel_message <-- this seems worst choice though, inspecting every message just in case it is a join/leave message.

@SwissalpS
Copy link
Contributor Author

Well, 3. seemed to be the easiest and fastest way to get this feature going, so #86

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants