Skip to content

fix: mapping of Komoot categories#440

Open
Mik- wants to merge 2 commits intoopen-wanderer:mainfrom
Mik-:komoot-import-categories
Open

fix: mapping of Komoot categories#440
Mik- wants to merge 2 commits intoopen-wanderer:mainfrom
Mik-:komoot-import-categories

Conversation

@Mik-
Copy link

@Mik- Mik- commented Aug 3, 2025

Add missing category mappings and fix sports.
Also do exact mapping, if matching user defined categories are present.

Fixes #439

Mik- added 2 commits August 3, 2025 12:29
Add missing category mappings and fix sports.
Also do exact mapping, if matching user defined categories are defined.
@Mik-
Copy link
Author

Mik- commented Aug 3, 2025

I do also a mapping to manually created categories, which fits better to the Komoot categories. A drawback is, that they must created exactly like defined in the code.

Maybe it's better to define a mapping table for this. And maybe this could be eligible to the Strava sync as well. But I think, the current version can help some guys and it would not break anything.

@cugu
Copy link
Collaborator

cugu commented Aug 3, 2025

Should the exact map map all values to unique values? Some people might want to tract e.g. E-MTB and MTB separatly.

@Mik-
Copy link
Author

Mik- commented Aug 3, 2025

The exact mapping is quit opinionated. There could/should be special E-Bike versions.
All tours, I've imported so far did not have a sport starting with e_.... I don't know how Komoot handle this.
I just took over all sports from the API docs. I also did not fiddle out how to create a tour on Komoot with sports marked as "Tracking sports type". Maybe this could be done via "Garmin watch to Komoot sync" or so.

As I'm pretty new Wanderer, I wanted to get the sync right with the default categories. IMHO the exact categories should be handled in a more user friendly way on the long run. But this is something, which concerns also other parts of the app. Nothing I want to propose as a newbie...

@slothful-vassal
Copy link
Contributor

Hello,

I like the idea of introducing a category mapping system. I don't know how @Flomp feels about it, but I personally don't think your approach is flexible enough. I hope you don't mind, but I've tried implementing a different, more generic approach (see #649).

Best regards and thank you very much for your commitment.

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.

Bug in mapping Komoot categories

3 participants