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 log ascent - counting of ascents incorrect/misleading #4085

Closed killakalle closed 2 years ago

killakalle commented 2 years ago

In the Release forum Simon wrote:

"If you have one attempt and it is successful then you should log 1. If you have 2 attempts and the second is successful then 2, etc."

To reproduce

  1. Log a climb with the new log interface
  2. Select redpoint - which is a successful ascent
  3. For "Number of attempts" select 2
  4. Save

Expected The confirmation modal reads: "You have 2 ascents. 1 successful and 1 failed."

Actual The confirmation modal reads: "You have 3 ascents. 1 successful and 2 failed."

Apparently, the ticked ascent, e.g. redpoint counts as 1. And the number of attempts counts as additionally failed ascents. Probably it should rather be

image

image

image

Link to ascent: https://www.thecrag.com/ascent/6191197554

scd commented 2 years ago

I have hot fixed this.

killakalle commented 2 years ago

@scd

I still think there is a need to clarify terminology or to update the confirmation message.

image

rouletout commented 2 years ago

Re-opened, looks like number of attempts is not taken into account if all failed?

scd commented 2 years ago

@killakalle thanks for re-analysing. I have fixed this in my dev. After next release please test again.