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

Add feature to automatically send checks when UT says they're in logic #89

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

silasary
Copy link
Collaborator

Alternative take on #74.
Not tested yet, do not merge.

@silasary silasary requested a review from axxroytovu September 28, 2024 14:47
@FuzzyGamesOn
Copy link
Collaborator

FuzzyGamesOn commented Sep 28, 2024

I don't think this should be a Game option. Features that auto-spam a multiworld without requiring user interaction first are probably a bad idea in general.

imo, this should always be an opt-in that's toggled on after connecting in the client. A button to toggle would be fine, or a command, either works. (And the ability to toggle it off, of course.)

@FuzzyGamesOn FuzzyGamesOn self-requested a review September 28, 2024 16:42
@silasary
Copy link
Collaborator Author

I made it a game option because LinkLink and Progressivizer pretty much always want it enabled, and that's what the original PR did

@nicopop
Copy link
Contributor

nicopop commented Sep 29, 2024

if we want to "hide" it we could always place it in the meta.json since the stuff in that file is more technical

Copy link
Contributor

@axxroytovu axxroytovu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm in favor of the current setup. It feels like allowing the game author to set the auto-send to "default on" which feels perfectly reasonable for the more meta uses of manual.

@FuzzyGamesOn
Copy link
Collaborator

We're not just designing a feature for a handful of meta apworlds to use. We're designing a feature that any Manual apworld can use. And this current design of the feature is one that could be enabled silently without the player of that world knowing.

That's really bad. And I'm shocked that I have to (again) mention that that's bad.

Also, this current design of the feature has the potential for large-scale location spam, which is a concern I brought up in the original feature suggestion thread for the feature: https://discord.com/channels/1097532591650910289/1181665955542212740/1181699938086817842

We have to consider the main potential risks of this feature when we're making it. Please revise the feature in a way that makes it safe for users and for multiworlds, or close the PR.

@axxroytovu
Copy link
Contributor

Clarifying question: what do you mean by "enabled silently without the player of that world knowing"?

Are you saying that people will enable autorelease on other peoples' worlds without their knowledge, or that the author enabling a feature is "without the player's knowledge"?

@axxroytovu
Copy link
Contributor

Is there a way to add a "two-factor" check similar to deathlink? I.e. t must be turned on both in the game options and the player yaml to be enabled by default. Or is that just not worth the extra hassle and it's better to have the player type /toggle_autosend every time?

@FuzzyGamesOn
Copy link
Collaborator

Clarifying question: what do you mean by "enabled silently without the player of that world knowing"?

Are you saying that people will enable autorelease on other peoples' worlds without their knowledge, or that the author enabling a feature is "without the player's knowledge"?

The latter: that the author enabling a feature is "without the player's knowledge".

If I'm a player, and a world creator releases an apworld for a game franchise that I know/enjoy, and I decide to take their apworld into a sync/async... What indication do I have (as a player) that the world creator enabled this option in their JSON configs? And that, when I connect for the first time, whatever they marked as open is going to get automatically sent into the multiworld I joined.

@axxroytovu axxroytovu self-requested a review October 1, 2024 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants