Skip to content

Conversation

@davight
Copy link
Contributor

@davight davight commented Nov 22, 2025

First part of updating sidebar command. Related discussion - https://discord.com/channels/315163488085475337/1439931889405395027

action = Action.valueOf(arg.getValue().toUpperCase());
}
else if (!scriptEntry.hasObject("title")
&& arg.matchesPrefix("title", "t", "objective", "obj", "o")) {
Copy link
Member

Choose a reason for hiding this comment

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

These were probably used to some degree as well & need some sort of back support, there's a addRemappedPrefixes util method you can call in the constructor (just make sure it works fine with the command code gen stuff because it's from before that and I don't remember offhand).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oops missed those, works fine with addRemappedPrefixes so should be fine now, also did this in command Title update PR

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.

2 participants