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

The first comma is replaced by a dot in string option #3874

Closed
Vicente015 opened this issue Sep 25, 2021 · 4 comments
Closed

The first comma is replaced by a dot in string option #3874

Vicente015 opened this issue Sep 25, 2021 · 4 comments
Labels
slash commands synced Synced to internal tracker

Comments

@Vicente015
Copy link

Description
When I have a string option and I enter numbers separated by commas, the first comma is always replaced by a dot.
It seems that the error does not occur in the API, but in the client that converts it when sending it.

Steps to Reproduce

  1. Have a string option
  2. Enter some numbers separated by commas (ej: 1,2,3) without whitespaces.
  3. Send the command and check the received interaction or the sent command info.

Expected Behavior
Doesn't convert the first comma into a dot

Current Behavior
It converts the first comma into a dot

Screenshots/Videos
vKR3QFh5mv

Interaction payload:
image

Client and System Information
Canary 99031 (720b494)
Host 1.0.42
Windows 11 64-Bit (10.0.22000)

@Diegomcha
Copy link

Diegomcha commented Sep 25, 2021

Can't reproduce

4hgQpciRf7

Client and System Information
Stable 99016 (0d5a801)
Host 1.0.9003
Windows 11 64-Bit (10.0.22000)

@Mehgugs
Copy link
Contributor

Mehgugs commented Sep 25, 2021

This only affects people whose discord language is set to a language which uses a comma as a decimal separator. This issue was reported in ddevs on 30th august and was apparently logged as a bug. No idea what happened afterwards and it seems like it's not fixed.

I can also reproduce this when using discord in French.

ddevs conversation is around here:
https://discord.com/channels/613425648685547541/788586647142793246/882016779453464656

@ilxlodev
Copy link
Contributor

ilxlodev commented Sep 27, 2021

I can reproduce this issue.

image

image

The bot also receives the data as "1. 2, 3" instead of "1, 2, 3".


Language: Spanish.
Canary: 99069 (b72fa3f).
Host: 1.0.42.
Windows 10 (64-Bit).

@typpo typpo added the synced Synced to internal tracker label Sep 27, 2021
@devsnek
Copy link
Contributor

devsnek commented Oct 12, 2021

A fix has been made and should be released soonish

@devsnek devsnek closed this as completed Oct 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
slash commands synced Synced to internal tracker
Projects
None yet
Development

No branches or pull requests

6 participants