Skip to content
New issue

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

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

Already on GitHub? # to your account

Update name command #1789

Open
wants to merge 5 commits into
base: development
Choose a base branch
from

Conversation

xdzaster
Copy link
Collaborator

@xdzaster xdzaster commented Jan 10, 2025

Changes

  1. Renamed the !name command to !changeName

    • The old command !name has been replaced with !changeName to better reflect its purpose and improve clarity. Users must now use !changeName instead of !name.
  2. Added a confirmation step to the !changeName command

    • To ensure users don't accidentally change their name, the !changeName command now requires a confirmation parameter.
    • Users must include i_am_sure=yes_i_am in the command for it to execute.
      For example:
      !changeName name=John&i_am_sure=yes_i_am
      
      Without this confirmation parameter, the command will not proceed.
  3. Updated command parsing to support the & character in values

    • The command handling logic has been improved to correctly parse values that contain the & character. For example, if a user inputs a parameter like name=Tom & Jerry, the bot will now handle it without treating & as a separator or causing errors. This ensures smoother handling of inputs.

Screenshot

image

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant