theCrag / website

theCrag.com: Add your voice and help guide the development of the world's largest collaborative rock climbing & bouldering platform
https://www.thecrag.com/
109 stars 8 forks source link

New ascent log - too many commas #4091

Closed killakalle closed 2 years ago

killakalle commented 2 years ago

Using the new ascent log, somehow there is a comma where there should be none - as if there was an additional (invisible) person. image

In the ascent details all looks fine to me image

Stream event: https://www.thecrag.com/event/6267194670

rouletout commented 2 years ago

This looks like a small bug

scd commented 2 years ago

I am not sure what happened here. I was unable to reproduce in my dev.

When I resaved your ascent the commas disappeared. Until we identify what caused the extra comma in the first place I don't think there is anything else we can do.

I am closing but please re-open or create a new issue if you discover how to repeat this bug.

skomaroff commented 2 years ago

I can reproduce this. Enter " ," (one space followed by a comma) for the climber's name. Two or more spaces works too.

The empty climber's name disappears when editting the route.

image

image

Essentially a duplicate of #3973

skomaroff commented 2 years ago

It's possible to get a lot more extra commas too:

image

scd commented 2 years ago

In dev I have fixed the storing of extra commas. For next release