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

Elementimprovements #166

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

kylepokorski
Copy link
Contributor

better version of #163
Currently:Adds variable IDs, suggestion result saving to "last" element, and making combination code use same getElementIDs function

Making combining use same getElementIds function as !ac does
Making getElementId call getElementIds
adding variable ids
it saves the voting result to the creator's combcache
Fixing plural for singular
Making suggesting by ID work
fixing 1 nonexistent element not being capitalized
@kylepokorski kylepokorski marked this pull request as draft February 13, 2025 02:59
Adding multiple *X (ex: *4 Water, *4 Air results in Water+Water+Water+Water+Air+Air+Air+Air)
Creating function ApplyMultiplier to deal with multiplier (this made it possible to add this feature)
Adding {text}, {raw} which make following a name and not an ID or multiplier
however {raw} is supposed to ignore separators too which is not implemented
Implementing {raw},making *N work with + as first element . Adding {name} as an alias of {text}. Adding {id} as alias of #
@kylepokorski kylepokorski marked this pull request as ready for review February 20, 2025 22:35
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.

1 participant