Tick Climbs and Show Stats

gardaholm commented 2 months ago

This PR addresses #46, #54, #55 and #73 For testing I've put this branch online on fly.to: climbdex.why.dev

Menu Elements and Result View

Dropdown / Hamburger Menu is shown to clean up the Result view a bit. If user is logged in there is an option to logout, tick climb and show stats. If user user is not logged in there is only the beta information available. Result List marks Sends in green and Attempts in yellow, to give a quick information about the status of the climb.

[ ] To support this view the Boardlib PR https://github.com/lemeryfertitta/BoardLib/pull/41 is needed to fetch and collect the logbook in the correct way. I've currently set my Branch in the requirements.txt for testing.

Simple Climbing and Attempt Stats

If the user is logged in, a separate menu item for Statistics is shown. This modal shows amount of attempts and sessions and when the last session was in a very basic form. I guess this place could be used for further improvements like suggested in Issue https://github.com/lemeryfertitta/Climbdex/issues/28

Tick Modal

Ticking climbs can be done via the new modal, which should be way faster than compared to the aurora app. Ticking a Climb is done via the boardlib function save_ascent. Difficulty is prefilled but can be changed. Number of attempts can be set when the slider is set to Send instead of Flash.

[ ] Only Ascents are currently possible, but it should be easy to hook up a save_attempt function and add a 3rd button into the modal (flash / send / attempt) as written here
[ ] For the bids and attempts in the api we currently use the same value, because it's unclear what fields are being used. But so far comments, tries, rating, comments, emojis, etc. are all shown correctly in the aurora apps.

gardaholm commented 2 months ago

updated aggregated_logbook() to use the aggregate=True parameter for the modified logbook_entries as written in this PR https://github.com/lemeryfertitta/BoardLib/pull/41

gardaholm commented 2 months ago

Thanks for the merge and your work! Hope its a good first addition for the save dialog and we could save you at least some time :) and thanks for refining the code 🙏

lemeryfertitta / Climbdex