Skip to content
This repository has been archived by the owner on Aug 8, 2024. It is now read-only.

Refactor bikelane categories, cleanup area code #137

Merged
merged 8 commits into from
May 14, 2024
Merged

Conversation

tordans
Copy link
Contributor

@tordans tordans commented May 13, 2024

  • refactor bikelane categories to use a more class like pattern
  • check and update comments per category (some are todo)
  • add some todos where things need to be checked
  • the area stuff was confusing and is cleaned up now

Ping https://github.com/FixMyBerlin/private-issues/issues/1664

@tordans tordans requested a review from rush42 May 13, 2024 10:38
@tordans tordans changed the base branch from main to develop May 13, 2024 10:38
@tordans tordans mentioned this pull request May 13, 2024
1 task
tordans and others added 2 commits May 14, 2024 13:19
Some of the comments here are from 2b135c8 and in fact wrong.

We want all living streets as bike friendly category. But we want to filter
1. when all vehicles except bikes are prohibited
2. when bikes are prohibited explicitly
@tordans tordans merged commit b984d53 into develop May 14, 2024
1 check passed
@tordans tordans deleted the bikelane-categories branch May 14, 2024 13:50
rush42 pushed a commit to FixMyBerlin/atlas-app that referenced this pull request Jul 15, 2024
…geo#137)

- refactor bikelane categories to use a more class like pattern
- check and update comments per category (some are todo)
- add some todos where things need to be checked
- the area stuff was confusing and is cleaned up now

Ping FixMyBerlin/private-issues#1664
rush42 pushed a commit to FixMyBerlin/atlas-app that referenced this pull request Jul 15, 2024
…geo#137)

- refactor bikelane categories to use a more class like pattern
- check and update comments per category (some are todo)
- add some todos where things need to be checked
- the area stuff was confusing and is cleaned up now

Ping FixMyBerlin/private-issues#1664
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants