Skip to content

WIP - Adding ascent speed (stats + graph)#1227

Open
redref wants to merge 5 commits intothomaschampagne:developfrom
redref:pr
Open

WIP - Adding ascent speed (stats + graph)#1227
redref wants to merge 5 commits intothomaschampagne:developfrom
redref:pr

Conversation

@redref
Copy link

@redref redref commented Aug 4, 2025

Hello !

To begin, thank you for this amazing work, from a long time trail runner user ! And new app is just amazing ! Kudos to you 😉

Mostly beginning to enter the codebase as a non-front developer my side. For trail running, it would be nice to have ascent speed, so I tried to integrate it quite successfully for my use.

Now, I need to package this in a neat way, but it seems testsuite are failing my side (even without those changes)

    Details:

    /home/<>/soft/elevate/desktop/node_modules/serialize-error/index.js:1
    ({"Object.<anonymous>":function(module,exports,require,__dirname,__filename,jest){import errorConstructors from './error-constructors.js';
                                                                                      ^^^^^^

    SyntaxError: Cannot use import statement outside a module

Any help / opinion / missing parts notice would be nice (and yeah, I am not good at JS internals 😉 ).

redref added 5 commits August 4, 2025 11:44
From my tests, works better than backoff as triggering a 429 seems to extend the lock period
and api usage is not well reported

not a neat fix but improve the overall process.
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