fix build errrors detected by sonarcloud

.forgejo/workflows/build-image.yml

@@ -1,20 +0,0 @@
-name: Build docker image
-on: [push]
-
-jobs:
-  build-image:
-    runs-on: ubuntu-latest
-    container:
-      image: catthehacker/ubuntu:act-latest
-    steps:
-      - name: Login to Forgejo docker registry
-        uses: docker/login-action@v3.0.0
-        with:
-          registry: git.pub.solar
-          username: hakkonaut
-          password: ${{ secrets.GIT_AUTH_TOKEN }}
-      - name: Build and push
-        uses: docker/build-push-action@v5.1.0
-        with:
-          push: true
-          tags: git.pub.solar/pub-solar/obs-portal:latest

.github/dependabot.yml

@@ -0,0 +1,19 @@
+# see also https://docs.github.com/en/code-security/supply-chain-security/keeping-your-dependencies-updated-automatically
+
+version: 2
+updates:
+  - package-ecosystem: "docker"
+    directory: "/"
+    schedule:
+      interval: "daily"
+
+# Disable submodules for the time being - they seem to not respect the branch of scripts we're using.
+#  - package-ecosystem: "gitsubmodule"
+#    directory: "/"
+#    schedule:
+#      interval: "daily"
+      
+  - package-ecosystem: "pip"
+    directory: "/api"
+    schedule:
+      interval: "daily"

CHANGELOG.md

@@ -1,38 +1,5 @@
 # Changelog
 
-## 0.8.1
-
-### Improvements
-
-* The zone (urban/rural) is now also exported with the events GeoJson export.
-
-### Bug Fixes
-
-* Update to a current version of gpstime (python dependency) fix portal startup.
-
-## 0.8.0
-
-### Features
-
-* Bulk actions on users owned tracks (reprocess, download, make private, make public, delete) (#269, #38)
-* Easy sorting by device for "multi-device users" (e.g. group lending out OBSes)
-* Region display at higher zoom levels to easily find interesting areas (#112)
-* Export of road statistics on top of the already-existing event statistics (#341)
-
-### Improvements
-
-* Refactored database access to hopefully combat portal crashes (#337)
-* New infrastructure for map imports that makes import of larger maps possible on small VMs (#334)
-* Reference current postgres and postgis versions in docker-compose.yaml files (#286)
-* Configurable terms-and-conditions link (#320)
-* French translation by @cbiteau (#303)
-
-### Bug Fixes
-
-* Logout not working (#285)
-* Duplicate road usage hashes (#335, #253)
-* cannot import name .... (#338)
-
 ## 0.7.0
 
 ### Features

Dockerfile

@@ -1,5 +1,44 @@
 # This dockerfile is for the API + Frontend production image
 
+#############################################
+# Build osm2pgsql AS builder
+#############################################
+
+# This image should be the same as final one, because of the lib versions
+FROM python:3.9.7-bullseye as osm2pgsql-builder
+
+ARG DEBIAN_FRONTEND=noninteractive
+ENV TZ=Europe/Berlin
+ENV OSM2PGSQL_VERSION=1.5.1
+
+# Dependencies
+RUN apt-get update &&\
+    apt-get install -y \
+    make \
+    cmake \
+    g++ \
+    libboost-dev \
+    libboost-system-dev \
+    libboost-filesystem-dev \
+    libexpat1-dev \
+    zlib1g-dev \
+    libbz2-dev \
+    libpq-dev \
+    libproj-dev \
+    lua5.3 \
+    liblua5.3-dev \
+    git &&\
+    rm -rf /var/lib/apt/lists/*
+
+# Clone & Build
+RUN git clone  --branch $OSM2PGSQL_VERSION https://github.com/openstreetmap/osm2pgsql.git &&\
+    cd osm2pgsql/ &&\
+    mkdir build &&\
+    cd build &&\
+    cmake .. &&\
+    make -j4 &&\
+    make install
+
 #############################################
 # Build the frontend AS builder
 #############################################
@@ -21,7 +60,7 @@ RUN npm run build
 # Build the API and add the built frontend to it
 #############################################
 
-FROM python:3.11.3-bullseye
+FROM python:3.9.7-bullseye
 
 RUN apt-get update &&\
     apt-get install -y \
@@ -54,7 +93,11 @@ ADD api/obs /opt/obs/api/obs/
 ADD api/tools /opt/obs/api/tools/
 RUN pip install -e /opt/obs/api/
 
+ADD roads_import.lua /opt/obs/api/tools
+ADD osm2pgsql.sh /opt/obs/api/tools
+
 COPY --from=frontend-builder /opt/obs/frontend/build /opt/obs/frontend/build
+COPY --from=osm2pgsql-builder /usr/local/bin/osm2pgsql /usr/local/bin/osm2pgsql
 
 EXPOSE 3000
 

README.md

@@ -36,11 +36,10 @@ git submodule update --init --recursive
 
 ## Production setup
 
-There is a guide for a deployment based on docker at
+There is a guide for a deployment based on docker in the
-[docs/production-deployment.md](docs/production-deployment.md). Lots of
+[deployment](deployment) folder. Lots of non-docker deployment strategy are
-non-docker deployment strategies are possible, but they are not "officially"
+possible, but they are not "officially" supported, so please do not expect the
-supported, so please do not expect the authors of the software to assist in
+authors of the software to assist in troubleshooting. 
-troubleshooting. 
 
 This is a rather complex application, and it is expected that you know the
 basics of deploying a modern web application securely onto a production server.
@@ -53,8 +52,7 @@ Please note that you will always need to install your own reverse proxy that
 terminates TLS for you and handles certificates. We do not support TLS directly
 in the application, instead, please use this prefered method. 
 
-Upgrading and migrating is described in [UPGRADING.md](./UPGRADING.md) for each
+Upgrading and migrating is descrube
-version.
 
 ### Migrating (Production)
 
@@ -77,6 +75,18 @@ docker-compose run --rm api alembic upgrade head
 docker-compose run --rm api tools/prepare_sql_tiles
 ```
 
+
+docker-compose run --rm api alembic upgrade head
+
+### Upgrading from v0.2 to v0.3
+
+After v0.2 we switched the underlying technology of the API and the database.
+We now have no more MongoDB, instead, everything has moved to the PostgreSQL
+installation. For development setups, it is advised to just reset the whole
+state (remove the `local` folder) and start fresh. For production upgrades,
+please follow the relevant section in [`UPGRADING.md`](./UPGRADING.md).
+
+
 ## Development setup
 
 We've moved the whole development setup into Docker to make it easy for
@@ -91,6 +101,7 @@ Then clone the repository as described above.
 
 ### Configure Keycloak
 
+
 Login will not be possible until you configure the keycloak realm correctly. Boot your keycloak instance:
 
 ```bash
@@ -153,7 +164,7 @@ You will need to re-run this command after updates, to migrate the database and
 (re-)create the functions in the SQL database that are used when generating
 vector tiles.
 
-You should also [import OpenStreetMap data](docs/osm-import.md) now.
+You should also import OpenStreetMap data now, see below for instructions.
 
 ### Boot the application
 
@@ -179,6 +190,48 @@ docker-compose run --rm api alembic upgrade head
 ```
 
 
+## Import OpenStreetMap data
+
+**Hint:** This step may be skipped if you are using [Lean mode](./docs/lean-mode.md).
+
+You need to import road information from OpenStreetMap for the portal to work.
+This information is stored in your PostgreSQL database and used when processing
+tracks (instead of querying the Overpass API), as well as for vector tile
+generation. The process applies to both development and production setups. For
+development, you should choose a small area for testing, such as your local
+county or city, to keep the amount of data small. For production use you have
+to import the whole region you are serving.
+
+* Install `osm2pgsql`. 
+* Download the area(s) you would like to import from [GeoFabrik](https://download.geofabrik.de). 
+* Import each file like this:
+
+    ```bash
+    osm2pgsql --create --hstore --style roads_import.lua -O flex \
+      -H localhost -d obs -U obs -W \
+      path/to/downloaded/myarea-latest.osm.pbf 
+    ```
+
+You might need to adjust the host, database and username (`-H`, `-d`, `-U`) to
+your setup, and also provide the correct password when queried. For the
+development setup the password is `obs`. For production, you might need to
+expose the containers port and/or create a TCP tunnel, for example with SSH,
+such that you can run the import from your local host and write to the remote
+database.
+
+The import process should take a few seconds to minutes, depending on the area
+size. A whole country might even take one or more hours. You should probably
+not try to import `planet.osm.pbf`. 
+
+You can run the process multiple times, with the same or different area files,
+to import or update the data. However, for this to work, the actual [command
+line arguments](https://osm2pgsql.org/doc/manual.html#running-osm2pgsql) are a
+bit different each time, including when first importing, and the disk space
+required is much higher. 
+
+Refer to the documentation of `osm2pgsql` for assistance. We are using "flex
+mode", the provided script `roads_import.lua` describes the transformations
+and extractions to perform on the original data.
 
 ## Troubleshooting
 

UPGRADING.md

@@ -1,43 +1,9 @@
 # Upgrading
+
 This document describes the general steps to upgrade between major changes.
 Simple migrations, e.g. for adding schema changes, are not documented
 explicitly. Their general usage is described in the [README](./README.md) (for
-development) and [docs/production-deployment.md](docs/production-deployment.md) (for production).
+development) and [deployment/README.md](deployment/README.md) (for production).
-
-
-## 0.8.1
- 
-- Get the release in your source folder (``git pull; git checkout 0.8.0`` and update submodules ``git submodule update --recursive``)
-- Rebuild images ``docker-compose build``
-- No database upgrade is required, but tile functions need an update:
-  ```bash
-  docker-compose run --rm portal tools/prepare_sql_tiles.py
-  ```
-- Start your portal and worker services. ``docker-compose up -d worker portal``
-
-
-## 0.8.0
-Upgrade to `0.7.x` first. See below for details. Then follow these steps:
-
-> **Warning** The update includes a reprocessing of tracks after import. Depending on the number of tracks this can take a few hours. The portal is reachable during that time but events disappear and incrementally reappear during reimport.
-
-> **Info** With this version the import process for OpenStreetMap data has changed: the [new process](docs/osm-import.md) is easier on resources and finally permits to import a full country on a low-end VM.
-
-- Do your [usual backup](docs/production-deployment.md)
-- get the release in your source folder (``git pull; git checkout 0.8.0`` and update submodules ``git submodule update --recursive``)
-- Rebuild images ``docker-compose build``
-- Stop your portal and worker services ``docker-compose stop worker portal``
-- run upgrade
-  ```bash
-  docker-compose run --rm portal tools/upgrade.py
-  ```
-  this automatically does the following
-  - Migration of database schema using alembic.
-  - Upgrade of SQL tile schema to new schema.
-  - Import the nuts-regions from the web into the database.
-  - Trigger a re-import of all tracks.
-- Start your portal and worker services. ``docker-compose up -d worker portal``
-
 
 ## 0.7.0
 
@@ -91,7 +57,7 @@ You can, but do not have to, reimport all tracks. This will generate a GPX file
 for each track and allow the users to download those. If a GPX file has not yet
 been created, the download will fail. To reimport all tracks, log in to your
 PostgreSQL database (instructions are in [README.md](./README.md) for
-development and [docs/production-deployment.md](./docs/production-deployment.md) for production)
+development and [deployment/README.md](./deployment/README.md) for production)
 and run:
 
 ```sql
@@ -111,7 +77,7 @@ Make sure your worker is running to process the queue.
   `POSTGRES_MAX_OVERFLOW`. Check the example config for sane default values.
 * Re-run `tools/prepare_sql_tiles.py` again (see README)
 * It has been made easier to import OSM data, check
-  [docs/production-deployment.md](./docs/production-deployment.md) for the sections "Download
+  [deployment/README.md](deployment/README.md) for the sections "Download
   OpenStreetMap maps" and "Import OpenStreetMap data". You can now download
   multiple .pbf files and then import them at once, using the docker image
   built with the `Dockerfile`. Alternatively, you can choose to enable [lean
@@ -166,5 +132,5 @@ Make sure your worker is running to process the queue.
   `export/users.json` into your realm, it will re-add all the users from the
   old installation. You should delete the file and `export/` folder afterwards.
 * Start `portal`.
-* Consider configuring a worker service. See [docs/production-deployment.md](./docs/production-deployment.md).
+* Consider configuring a worker service. See [deployment/README.md](deployment/README.md).
 

api/Dockerfile

@@ -1,4 +1,4 @@
-FROM python:3.11.3-bullseye
+FROM python:3.9.7-bullseye
 
 WORKDIR /opt/obs/api
 

api/config.dev.py

@@ -2,8 +2,9 @@ HOST = "0.0.0.0"
 PORT = 3000
 DEBUG = True
 VERBOSE = False
-AUTO_RELOAD = True
+AUTO_RESTART = True
 SECRET = "!!!!!!!!!!!!CHANGE ME!!!!!!!!!!!!"
+LEAN_MODE = False
 POSTGRES_URL = "postgresql+asyncpg://obs:obs@postgres/obs"
 POSTGRES_POOL_SIZE = 20
 POSTGRES_MAX_OVERFLOW = 2 * POSTGRES_POOL_SIZE
@@ -18,7 +19,6 @@ FRONTEND_DIR = None
 FRONTEND_CONFIG = {
     "imprintUrl": "https://example.com/imprint",
     "privacyPolicyUrl": "https://example.com/privacy",
-    # "termsUrl": "https://example.com/terms", # Link is only shown when set
     "mapHome": {"zoom": 6, "longitude": 10.2, "latitude": 51.3},
     # "banner": {"text": "This is a development installation.", "style": "info"},
 }
@@ -29,7 +29,5 @@ ADDITIONAL_CORS_ORIGINS = [
     "http://localhost:8880/",  # for maputnik on 8880
     "http://localhost:8888/",  # for maputnik on 8888
 ]
-TILE_SEMAPHORE_SIZE = 4
-EXPORT_SEMAPHORE_SIZE = 4
 
 # vim: set ft=python :

api/config.py.example

@@ -5,7 +5,12 @@ PORT = 3000
 # Extended log output, but slower
 DEBUG = False
 VERBOSE = DEBUG
-AUTO_RELOAD = DEBUG
+AUTO_RESTART = DEBUG
+
+# Turn on lean mode to simplify the setup. Lots of features will be
+# unavailable, but you will not need to manage OpenStreetMap data. Please make
+# sure to configure the OBS_FACE_CACHE_DIR correctly for lean mode.
+LEAN_MODE = False
 
 # Required to encrypt or sign sessions, cookies, tokens, etc.
 SECRET = "!!!<<<CHANGEME>>>!!!"
@@ -39,7 +44,6 @@ FRONTEND_DIR = "../frontend/build/"
 FRONTEND_CONFIG = {
     "imprintUrl": "https://example.com/imprint",
     "privacyPolicyUrl": "https://example.com/privacy",
-    # "termsUrl": "https://example.com/user_terms_and_conditions", # Link is only shown when set
     "mapHome": {"zoom": 6, "longitude": 10.2, "latitude": 51.3},
     "banner": {"text": "This is a test installation.", "style": "warning"},
 }
@@ -61,13 +65,4 @@ TILES_FILE = None
 # default. Python list, or whitespace separated string.
 ADDITIONAL_CORS_ORIGINS = None
 
-# How many asynchronous requests may be sent to the database to generate tile
-# information. Should be less than POSTGRES_POOL_SIZE to leave some connections
-# to the other features of the API ;)
-TILE_SEMAPHORE_SIZE = 4
-
-# How many asynchronous requests may generate exported data simultaneously.
-# Keep this small.
-EXPORT_SEMAPHORE_SIZE = 1
-
 # vim: set ft=python :

api/migrations/versions/35e7f1768f9b_create_table_road.py

@@ -22,16 +22,13 @@ def upgrade():
     op.create_table(
         "road",
         sa.Column(
-            "way_id", sa.BIGINT, primary_key=True, index=True, autoincrement=False
+            "way_id", sa.BIGINT, autoincrement=True, primary_key=True, index=True
         ),
         sa.Column("zone", dbtype("zone_type")),
-        sa.Column("name", sa.Text),
+        sa.Column("name", sa.String),
-        sa.Column("geometry", dbtype("geometry(LINESTRING,3857)")),
+        sa.Column("geometry", dbtype("GEOMETRY"), index=True),
         sa.Column("directionality", sa.Integer),
-        sa.Column("oneway", sa.Boolean),
+        sa.Column("oenway", sa.Boolean),
-    )
-    op.execute(
-        "CREATE INDEX road_geometry_idx ON road USING GIST (geometry) WITH (FILLFACTOR=100);"
     )
 
 

api/migrations/versions/587e69ecb466_transform_overtaking_event_geometry_to_.py

@@ -1,30 +0,0 @@
-"""transform overtaking_event geometry to 3857
-
-Revision ID: 587e69ecb466
-Revises: f4b0f460254d
-Create Date: 2023-04-01 14:30:49.927505
-
-"""
-from alembic import op
-import sqlalchemy as sa
-
-
-# revision identifiers, used by Alembic.
-revision = "587e69ecb466"
-down_revision = "f4b0f460254d"
-branch_labels = None
-depends_on = None
-
-
-def upgrade():
-    op.execute("UPDATE overtaking_event SET geometry = ST_Transform(geometry, 3857);")
-    op.execute(
-        "ALTER TABLE overtaking_event ALTER COLUMN geometry TYPE geometry(POINT, 3857);"
-    )
-
-
-def downgrade():
-    op.execute(
-        "ALTER TABLE overtaking_event ALTER COLUMN geometry TYPE geometry;"
-    )
-    op.execute("UPDATE overtaking_event SET geometry = ST_Transform(geometry, 4326);")

api/migrations/versions/7868aed76122_add_overtaking_event_index.py

@@ -1,26 +0,0 @@
-"""add_overtaking_event_index
-
-
-Revision ID: 7868aed76122
-Revises: 587e69ecb466
-Create Date: 2023-07-16 13:37:17.694079
-
-"""
-from alembic import op
-import sqlalchemy as sa
-
-
-# revision identifiers, used by Alembic.
-revision = '7868aed76122'
-down_revision = '587e69ecb466'
-branch_labels = None
-depends_on = None
-
-
-def upgrade():
-    op.execute("CREATE INDEX IF NOT EXISTS ix_overtaking_event_geometry ON overtaking_event using GIST(geometry);")
-
-
-def downgrade():
-    op.drop_index("ix_overtaking_event_geometry")
-

api/migrations/versions/a049e5eb24dd_create_table_region.py

@@ -21,13 +21,13 @@ depends_on = None
 def upgrade():
     op.create_table(
         "region",
-        sa.Column("id", sa.String(24), primary_key=True, index=True),
+        sa.Column(
-        sa.Column("name", sa.Text),
+            "relation_id", sa.BIGINT, autoincrement=True, primary_key=True, index=True
-        sa.Column("geometry", dbtype("GEOMETRY(GEOMETRY,3857)"), index=False),
+        ),
+        sa.Column("name", sa.String),
+        sa.Column("geometry", dbtype("GEOMETRY"), index=True),
         sa.Column("admin_level", sa.Integer, index=True),
-    )
+        sa.Column("tags", dbtype("HSTORE")),
-    op.execute(
-        "CREATE INDEX region_geometry_idx ON region USING GIST (geometry) WITH (FILLFACTOR=100);"
     )
 
 

api/migrations/versions/b8b0fbae50a4_add_import_groups.py

@@ -1,39 +0,0 @@
-"""add import groups
-
-Revision ID: b8b0fbae50a4
-Revises: f7b21148126a
-Create Date: 2023-03-26 09:41:36.621203
-
-"""
-from alembic import op
-import sqlalchemy as sa
-
-
-# revision identifiers, used by Alembic.
-revision = "b8b0fbae50a4"
-down_revision = "f7b21148126a"
-branch_labels = None
-depends_on = None
-
-
-def upgrade():
-    op.add_column(
-        "road",
-        sa.Column("import_group", sa.String(), nullable=True),
-    )
-    op.add_column(
-        "region",
-        sa.Column("import_group", sa.String(), nullable=True),
-    )
-
-    # Set existing to "osm2pgsql"
-    road = sa.table("road", sa.column("import_group", sa.String))
-    op.execute(road.update().values(import_group="osm2pgsql"))
-
-    region = sa.table("region", sa.column("import_group", sa.String))
-    op.execute(region.update().values(import_group="osm2pgsql"))
-
-
-def downgrade():
-    op.drop_column("road", "import_group")
-    op.drop_column("region", "import_group")

api/migrations/versions/f4b0f460254d_add_osm_id_indexes.py

@@ -1,24 +0,0 @@
-"""add osm id indexes
-
-Revision ID: f4b0f460254d
-Revises: b8b0fbae50a4
-Create Date: 2023-03-30 10:56:22.066768
-
-"""
-from alembic import op
-import sqlalchemy as sa
-
-
-# revision identifiers, used by Alembic.
-revision = "f4b0f460254d"
-down_revision = "b8b0fbae50a4"
-branch_labels = None
-depends_on = None
-
-
-def upgrade():
-    op.execute("CREATE INDEX IF NOT EXISTS ix_road_way_id ON road (way_id);")
-
-
-def downgrade():
-    op.drop_index("ix_road_way_id")

api/migrations/versions/f7b21148126a_add_user_device.py

@@ -1,41 +0,0 @@
-"""add user_device
-
-Revision ID: f7b21148126a
-Revises: a9627f63fbed
-Create Date: 2022-09-15 17:48:06.764342
-
-"""
-from alembic import op
-import sqlalchemy as sa
-
-
-# revision identifiers, used by Alembic.
-revision = "f7b21148126a"
-down_revision = "a049e5eb24dd"
-branch_labels = None
-depends_on = None
-
-
-def upgrade():
-    op.create_table(
-        "user_device",
-        sa.Column("id", sa.Integer, autoincrement=True, primary_key=True),
-        sa.Column("user_id", sa.Integer, sa.ForeignKey("user.id", ondelete="CASCADE")),
-        sa.Column("identifier", sa.String, nullable=False),
-        sa.Column("display_name", sa.String, nullable=True),
-        sa.Index("user_id_identifier", "user_id", "identifier", unique=True),
-    )
-    op.add_column(
-        "track",
-        sa.Column(
-            "user_device_id",
-            sa.Integer,
-            sa.ForeignKey("user_device.id", ondelete="RESTRICT"),
-            nullable=True,
-        ),
-    )
-
-
-def downgrade():
-    op.drop_column("track", "user_device_id")
-    op.drop_table("user_device")

api/obs/api/__init__.py

@@ -1 +1 @@
-__version__ = "0.8.1"
+__version__ = "0.7.0"

api/obs/api/app.py

@@ -1,4 +1,3 @@
-import asyncio
 import logging
 import re
 
@@ -22,60 +21,16 @@ from sqlalchemy import select
 from sqlalchemy.ext.asyncio import AsyncSession
 
 from obs.api.db import User, make_session, connect_db
-from obs.api.cors import setup_options, add_cors_headers
 from obs.api.utils import get_single_arg
+from sqlalchemy.util import asyncio
 
 log = logging.getLogger(__name__)
 
-
-class SanicAccessMessageFilter(logging.Filter):
-    """
-    A filter that modifies the log message of a sanic.access log entry to
-    include useful information.
-    """
-
-    def filter(self, record):
-        record.msg = f"{record.request} -> {record.status}"
-        return True
-
-
-def configure_sanic_logging():
-    for logger_name in ["sanic.root", "sanic.access", "sanic.error"]:
-        logger = logging.getLogger(logger_name)
-        for handler in logger.handlers:
-            logger.removeHandler(handler)
-
-    logger = logging.getLogger("sanic.access")
-    for filter_ in logger.filters:
-        logger.removeFilter(filter_)
-    logger.addFilter(SanicAccessMessageFilter())
-    logging.getLogger("sanic.root").setLevel(logging.WARNING)
-
-
 app = Sanic(
     "openbikesensor-api",
     env_prefix="OBS_",
+    log_config={},
 )
-configure_sanic_logging()
-
-app.config.update(
-    dict(
-        DEBUG=False,
-        VERBOSE=False,
-        AUTO_RELOAD=False,
-        POSTGRES_POOL_SIZE=20,
-        POSTGRES_MAX_OVERFLOW=40,
-        DEDICATED_WORKER=True,
-        FRONTEND_URL=None,
-        FRONTEND_HTTPS=True,
-        TILES_FILE=None,
-        TILE_SEMAPHORE_SIZE=4,
-        EXPORT_SEMAPHORE_SIZE=1,
-    )
-)
-
-# overwrite from defaults again
-app.config.load_environment_vars("OBS_")
 
 if isfile("./config.py"):
     app.update_config("./config.py")
@@ -104,39 +59,6 @@ class NoConnectionLostFilter(logging.Filter):
 logging.getLogger("sanic.error").addFilter(NoConnectionLostFilter)
 
 
-def setup_cors(app):
-    frontend_url = app.config.get("FRONTEND_URL")
-    additional_origins = app.config.get("ADDITIONAL_CORS_ORIGINS")
-    if not frontend_url and not additional_origins:
-        # No CORS configured
-        return
-
-    origins = []
-    if frontend_url:
-        u = urlparse(frontend_url)
-        origins.append(f"{u.scheme}://{u.netloc}")
-
-    if isinstance(additional_origins, str):
-        origins += re.split(r"\s+", additional_origins)
-    elif isinstance(additional_origins, list):
-        origins += additional_origins
-    elif additional_origins is not None:
-        raise ValueError(
-            "invalid option type for ADDITIONAL_CORS_ORIGINS, must be list or space separated str"
-        )
-
-    app.ctx.cors_origins = origins
-
-    # Add OPTIONS handlers to any route that is missing it
-    app.register_listener(setup_options, "before_server_start")
-
-    # Fill in CORS headers
-    app.register_middleware(add_cors_headers, "response")
-
-
-setup_cors(app)
-
-
 @app.exception(SanicException, BaseException)
 async def _handle_sanic_errors(_request, exception):
     if isinstance(exception, asyncio.CancelledError):
@@ -173,6 +95,39 @@ def configure_paths(c):
 configure_paths(app.config)
 
 
+def setup_cors(app):
+    frontend_url = app.config.get("FRONTEND_URL")
+    additional_origins = app.config.get("ADDITIONAL_CORS_ORIGINS")
+    if not frontend_url and not additional_origins:
+        # No CORS configured
+        return
+
+    origins = []
+    if frontend_url:
+        u = urlparse(frontend_url)
+        origins.append(f"{u.scheme}://{u.netloc}")
+
+    if isinstance(additional_origins, str):
+        origins += re.split(r"\s+", additional_origins)
+    elif isinstance(additional_origins, list):
+        origins += additional_origins
+    elif additional_origins is not None:
+        raise ValueError(
+            "invalid option type for ADDITIONAL_CORS_ORIGINS, must be list or space separated str"
+        )
+
+    from sanic_cors import CORS
+
+    CORS(
+        app,
+        origins=origins,
+        supports_credentials=True,
+        expose_headers={"Content-Disposition"},
+    )
+
+
+setup_cors(app)
+
 # TODO: use a different interface, maybe backed by the PostgreSQL, to allow
 # scaling the API
 Session(app, interface=InMemorySessionInterface())
@@ -187,12 +142,6 @@ async def app_connect_db(app, loop):
     )
     app.ctx._db_engine = await app.ctx._db_engine_ctx.__aenter__()
 
-    if app.config.TILE_SEMAPHORE_SIZE:
-        app.ctx.tile_semaphore = asyncio.Semaphore(app.config.TILE_SEMAPHORE_SIZE)
-
-    if app.config.EXPORT_SEMAPHORE_SIZE:
-        app.ctx.export_semaphore = asyncio.Semaphore(app.config.EXPORT_SEMAPHORE_SIZE)
-
 
 @app.after_server_stop
 async def app_disconnect_db(app, loop):
@@ -345,7 +294,9 @@ from .routes import (
     exports,
 )
 
+if not app.config.LEAN_MODE:
     from .routes import tiles, mapdetails
+
 from .routes import frontend
 
 

api/obs/api/cors.py

@@ -1,68 +0,0 @@
-from collections import defaultdict
-from typing import Dict, FrozenSet, Iterable
-
-from sanic import Sanic, response
-from sanic_routing.router import Route
-
-
-def _add_cors_headers(request, response, methods: Iterable[str]) -> None:
-    allow_methods = list(set(methods))
-
-    if "OPTIONS" not in allow_methods:
-        allow_methods.append("OPTIONS")
-
-    origin = request.headers.get("origin")
-    if origin in request.app.ctx.cors_origins:
-        headers = {
-            "Access-Control-Allow-Methods": ",".join(allow_methods),
-            "Access-Control-Allow-Origin": origin,
-            "Access-Control-Allow-Credentials": "true",
-            "Access-Control-Allow-Headers": (
-                "origin, content-type, accept, "
-                "authorization, x-xsrf-token, x-request-id"
-            ),
-            "Access-Control-Expose-Headers": "content-disposition",
-        }
-        response.headers.extend(headers)
-
-
-def add_cors_headers(request, response):
-    if request.method != "OPTIONS":
-        methods = [method for method in request.route.methods]
-        _add_cors_headers(request, response, methods)
-
-
-def _compile_routes_needing_options(routes: Dict[str, Route]) -> Dict[str, FrozenSet]:
-    needs_options = defaultdict(list)
-    # This is 21.12 and later. You will need to change this for older versions.
-    for route in routes.values():
-        if "OPTIONS" not in route.methods:
-            needs_options[route.uri].extend(route.methods)
-
-    return {uri: frozenset(methods) for uri, methods in dict(needs_options).items()}
-
-
-def _options_wrapper(handler, methods):
-    def wrapped_handler(request, *args, **kwargs):
-        nonlocal methods
-        return handler(request, methods)
-
-    return wrapped_handler
-
-
-async def options_handler(request, methods) -> response.HTTPResponse:
-    resp = response.empty()
-    _add_cors_headers(request, resp, methods)
-    return resp
-
-
-def setup_options(app: Sanic, _):
-    app.router.reset()
-    needs_options = _compile_routes_needing_options(app.router.routes_all)
-    for uri, methods in needs_options.items():
-        app.add_route(
-            _options_wrapper(options_handler, methods),
-            uri,
-            methods=["OPTIONS"],
-        )
-    app.router.finalize()

api/obs/api/db.py

@@ -34,9 +34,8 @@ from sqlalchemy import (
     select,
     text,
     literal,
-    Text,
 )
-from sqlalchemy.dialects.postgresql import UUID
+from sqlalchemy.dialects.postgresql import HSTORE, UUID
 
 
 log = logging.getLogger(__name__)
@@ -108,28 +107,6 @@ class Geometry(UserDefinedType):
         return func.ST_AsGeoJSON(func.ST_Transform(col, 4326), type_=self)
 
 
-class LineString(UserDefinedType):
-    def get_col_spec(self):
-        return "geometry(LineString, 3857)"
-
-    def bind_expression(self, bindvalue):
-        return func.ST_GeomFromGeoJSON(bindvalue, type_=self)
-
-    def column_expression(self, col):
-        return func.ST_AsGeoJSON(func.ST_Transform(col, 4326), type_=self)
-
-
-class GeometryGeometry(UserDefinedType):
-    def get_col_spec(self):
-        return "geometry(GEOMETRY, 3857)"
-
-    def bind_expression(self, bindvalue):
-        return func.ST_GeomFromGeoJSON(bindvalue, type_=self)
-
-    def column_expression(self, col):
-        return func.ST_AsGeoJSON(func.ST_Transform(col, 4326), type_=self)
-
-
 class OvertakingEvent(Base):
     __tablename__ = "overtaking_event"
     __table_args__ = (Index("road_segment", "way_id", "direction_reversed"),)
@@ -157,23 +134,12 @@ class OvertakingEvent(Base):
 
 class Road(Base):
     __tablename__ = "road"
-    way_id = Column(BIGINT, primary_key=True, index=True, autoincrement=False)
+    way_id = Column(BIGINT, primary_key=True, index=True)
     zone = Column(ZoneType)
-    name = Column(Text)
+    name = Column(String)
-    geometry = Column(LineString)
+    geometry = Column(Geometry)
     directionality = Column(Integer)
     oneway = Column(Boolean)
-    import_group = Column(String)
-
-    __table_args__ = (
-        # We keep the index name as osm2pgsql created it, way back when.
-        Index(
-            "road_geometry_idx",
-            "geometry",
-            postgresql_using="gist",
-            postgresql_with={"fillfactor": 100},
-        ),
-    )
 
     def to_dict(self):
         return {
@@ -200,12 +166,6 @@ class RoadUsage(Base):
     def __repr__(self):
         return f"<RoadUsage {self.id}>"
 
-    def __hash__(self):
-        return int(self.hex_hash, 16)
-
-    def __eq__(self, other):
-        return self.hex_hash == other.hex_hash
-
 
 NOW = text("NOW()")
 
@@ -261,12 +221,6 @@ class Track(Base):
         Integer, ForeignKey("user.id", ondelete="CASCADE"), nullable=False
     )
 
-    user_device_id = Column(
-        Integer,
-        ForeignKey("user_device.id", ondelete="RESTRICT"),
-        nullable=True,
-    )
-
     # Statistics... maybe we'll drop some of this if we can easily compute them from SQL
     recorded_at = Column(DateTime)
     recorded_until = Column(DateTime)
@@ -299,7 +253,6 @@ class Track(Base):
         if for_user_id is not None and for_user_id == self.author_id:
             result["uploadedByUserAgent"] = self.uploaded_by_user_agent
             result["originalFileName"] = self.original_file_name
-            result["userDeviceId"] = self.user_device_id
 
         if self.author:
             result["author"] = self.author.to_dict(for_user_id=for_user_id)
@@ -456,28 +409,6 @@ class User(Base):
         self.username = new_name
 
 
-class UserDevice(Base):
-    __tablename__ = "user_device"
-    id = Column(Integer, autoincrement=True, primary_key=True)
-    user_id = Column(Integer, ForeignKey("user.id", ondelete="CASCADE"))
-    identifier = Column(String, nullable=False)
-    display_name = Column(String, nullable=True)
-
-    __table_args__ = (
-        Index("user_id_identifier", "user_id", "identifier", unique=True),
-    )
-
-    def to_dict(self, for_user_id=None):
-        if for_user_id != self.user_id:
-            return {}
-
-        return {
-            "id": self.id,
-            "identifier": self.identifier,
-            "displayName": self.display_name,
-        }
-
-
 class Comment(Base):
     __tablename__ = "comment"
     id = Column(Integer, autoincrement=True, primary_key=True)
@@ -504,21 +435,11 @@ class Comment(Base):
 class Region(Base):
     __tablename__ = "region"
 
-    id = Column(String(24), primary_key=True, index=True)
+    relation_id = Column(BIGINT, primary_key=True, index=True)
-    name = Column(Text)
+    name = Column(String)
-    geometry = Column(GeometryGeometry)
+    geometry = Column(Geometry)
-    admin_level = Column(Integer, index=True)
+    admin_level = Column(Integer)
-    import_group = Column(String)
+    tags = Column(HSTORE)
-
-    __table_args__ = (
-        # We keep the index name as osm2pgsql created it, way back when.
-        Index(
-            "region_geometry_idx",
-            "geometry",
-            postgresql_using="gist",
-            postgresql_with={"fillfactor": 100},
-        ),
-    )
 
 
 Comment.author = relationship("User", back_populates="authored_comments")
@@ -547,14 +468,6 @@ Track.overtaking_events = relationship(
     passive_deletes=True,
 )
 
-Track.user_device = relationship("UserDevice", back_populates="tracks")
-UserDevice.tracks = relationship(
-    "Track",
-    order_by=Track.created_at,
-    back_populates="user_device",
-    passive_deletes=False,
-)
-
 
 # 0..4 Night, 4..10 Morning, 10..14 Noon, 14..18 Afternoon, 18..22 Evening, 22..00 Night
 # Two hour intervals

api/obs/api/process.py

@@ -8,7 +8,7 @@ import pytz
 from os.path import join
 from datetime import datetime
 
-from sqlalchemy import delete, func, select, and_
+from sqlalchemy import delete, select
 from sqlalchemy.orm import joinedload
 
 from obs.face.importer import ImportMeasurementsCsv
@@ -25,9 +25,9 @@ from obs.face.filter import (
     RequiredFieldsFilter,
 )
 
-from obs.face.osm import DataSource, DatabaseTileSource
+from obs.face.osm import DataSource, DatabaseTileSource, OverpassTileSource
 
-from obs.api.db import OvertakingEvent, RoadUsage, Track, UserDevice, make_session
+from obs.api.db import OvertakingEvent, RoadUsage, Track, make_session
 from obs.api.app import app
 
 log = logging.getLogger(__name__)
@@ -39,7 +39,12 @@ def get_data_source():
     mode, the OverpassTileSource is used to fetch data on demand. In normal
     mode, the roads database is used.
     """
-    return DataSource(DatabaseTileSource())
+    if app.config.LEAN_MODE:
+        tile_source = OverpassTileSource(cache_dir=app.config.OBS_FACE_CACHE_DIR)
+    else:
+        tile_source = DatabaseTileSource()
+
+    return DataSource(tile_source)
 
 
 async def process_tracks_loop(delay):
@@ -139,11 +144,10 @@ async def process_track(session, track, data_source):
         os.makedirs(output_dir, exist_ok=True)
 
         log.info("Annotating and filtering CSV file")
-        imported_data, statistics, track_metadata = ImportMeasurementsCsv().read(
+        imported_data, statistics = ImportMeasurementsCsv().read(
             original_file_path,
             user_id="dummy",  # TODO: user username or id or nothing?
             dataset_id=Track.slug,  # TODO: use track id or slug or nothing?
-            return_metadata=True,
         )
 
         annotator = AnnotateMeasurements(
@@ -213,36 +217,6 @@ async def process_track(session, track, data_source):
         await clear_track_data(session, track)
         await session.commit()
 
-        device_identifier = track_metadata.get("DeviceId")
-        if device_identifier:
-            if isinstance(device_identifier, list):
-                device_identifier = device_identifier[0]
-
-            log.info("Finding or creating device %s", device_identifier)
-            user_device = (
-                await session.execute(
-                    select(UserDevice).where(
-                        and_(
-                            UserDevice.user_id == track.author_id,
-                            UserDevice.identifier == device_identifier,
-                        )
-                    )
-                )
-            ).scalar()
-
-            log.debug("user_device is %s", user_device)
-
-            if not user_device:
-                user_device = UserDevice(
-                    user_id=track.author_id, identifier=device_identifier
-                )
-                log.debug("Create new device for this user")
-                session.add(user_device)
-
-            track.user_device = user_device
-        else:
-            log.info("No DeviceId in track metadata.")
-
         log.info("Import events into database...")
         await import_overtaking_events(session, track, overtaking_events)
 
@@ -306,16 +280,11 @@ async def import_overtaking_events(session, track, overtaking_events):
             hex_hash=hex_hash,
             way_id=m.get("OSM_way_id"),
             direction_reversed=m.get("OSM_way_orientation", 0) < 0,
-            geometry=func.ST_Transform(
+            geometry=json.dumps(
-                func.ST_GeomFromGeoJSON(
-                    json.dumps(
                 {
                     "type": "Point",
                     "coordinates": [m["longitude"], m["latitude"]],
                 }
-                    )
-                ),
-                3857,
             ),
             latitude=m["latitude"],
             longitude=m["longitude"],

api/obs/api/routes/exports.py

@@ -3,22 +3,15 @@ from enum import Enum
 from contextlib import contextmanager
 import zipfile
 import io
-import re
-import math
 from sqlite3 import connect
 
 import shapefile
 from obs.api.db import OvertakingEvent
-from sqlalchemy import select, func, text
+from sqlalchemy import select, func
 from sanic.response import raw
 from sanic.exceptions import InvalidUsage
 
 from obs.api.app import api, json as json_response
-from obs.api.utils import use_request_semaphore
-
-import logging
-
-log = logging.getLogger(__name__)
 
 
 class ExportFormat(str, Enum):
@@ -33,7 +26,7 @@ def parse_bounding_box(input_string):
             func.ST_Point(left, bottom),
             func.ST_Point(right, top),
         ),
-        4326,
+        3857,
     )
 
 
@@ -45,11 +38,11 @@ PROJECTION_4326 = (
 
 
 @contextmanager
-def shapefile_zip(shape_type=shapefile.POINT, basename="events"):
+def shapefile_zip():
     zip_buffer = io.BytesIO()
     shp, shx, dbf = (io.BytesIO() for _ in range(3))
     writer = shapefile.Writer(
-        shp=shp, shx=shx, dbf=dbf, shapeType=shape_type, encoding="utf8"
+        shp=shp, shx=shx, dbf=dbf, shapeType=shapefile.POINT, encoding="utf8"
     )
 
     yield writer, zip_buffer
@@ -58,68 +51,42 @@ def shapefile_zip(shape_type=shapefile.POINT, basename="events"):
     writer.close()
 
     zip_file = zipfile.ZipFile(zip_buffer, "a", zipfile.ZIP_DEFLATED, False)
-    zip_file.writestr(f"{basename}.shp", shp.getbuffer())
+    zip_file.writestr("events.shp", shp.getbuffer())
-    zip_file.writestr(f"{basename}.shx", shx.getbuffer())
+    zip_file.writestr("events.shx", shx.getbuffer())
-    zip_file.writestr(f"{basename}.dbf", dbf.getbuffer())
+    zip_file.writestr("events.dbf", dbf.getbuffer())
-    zip_file.writestr(f"{basename}.prj", PROJECTION_4326)
+    zip_file.writestr("events.prj", PROJECTION_4326)
     zip_file.close()
 
 
 @api.get(r"/export/events")
 async def export_events(req):
-    async with use_request_semaphore(req, "export_semaphore", timeout=30):
+    bbox = req.ctx.get_single_arg(
-        bbox = req.ctx.get_single_arg("bbox", default="-180,-90,180,90")
+        "bbox", default="-180,-90,180,90", convert=parse_bounding_box
-        assert re.match(r"(-?\d+\.?\d+,?){4}", bbox)
+    )
-        bbox = list(map(float, bbox.split(",")))
-
     fmt = req.ctx.get_single_arg("fmt", convert=ExportFormat)
 
-        events = await req.ctx.db.stream(
+    events = await req.ctx.db.stream_scalars(
-            text(
+        select(OvertakingEvent).where(OvertakingEvent.geometry.bool_op("&&")(bbox))
-                """
-                SELECT
-                    ST_AsGeoJSON(ST_Transform(geometry, 4326)) AS geometry,
-                    distance_overtaker,
-                    distance_stationary,
-                    way_id,
-                    direction,
-                    speed,
-                    time_stamp,
-                    course,
-                    zone
-                FROM
-                    layer_obs_events(
-                        ST_Transform(ST_MakeEnvelope(:bbox0, :bbox1, :bbox2, :bbox3, 4326), 3857),
-                        19,
-                        NULL,
-                        '1900-01-01'::timestamp,
-                        '2100-01-01'::timestamp
-                    )
-                """
-            ).bindparams(bbox0=bbox[0], bbox1=bbox[1], bbox2=bbox[2], bbox3=bbox[3])
     )
 
     if fmt == ExportFormat.SHAPEFILE:
-            with shapefile_zip(basename="events") as (writer, zip_buffer):
+        with shapefile_zip() as (writer, zip_buffer):
             writer.field("distance_overtaker", "N", decimal=4)
             writer.field("distance_stationary", "N", decimal=4)
             writer.field("way_id", "N", decimal=0)
             writer.field("direction", "N", decimal=0)
             writer.field("course", "N", decimal=4)
             writer.field("speed", "N", decimal=4)
-                writer.field("zone", "C")
 
             async for event in events:
-                    coords = json.loads(event.geometry)["coordinates"]
+                writer.point(event.longitude, event.latitude)
-                    writer.point(*coords)
                 writer.record(
                     distance_overtaker=event.distance_overtaker,
                     distance_stationary=event.distance_stationary,
-                        direction=event.direction,
+                    direction=-1 if event.direction_reversed else 1,
                     way_id=event.way_id,
                     course=event.course,
                     speed=event.speed,
-                        zone=event.zone
                     # "time"=event.time,
                 )
 
@@ -128,129 +95,18 @@ async def export_events(req):
     if fmt == ExportFormat.GEOJSON:
         features = []
         async for event in events:
-                geom = json.loads(event.geometry)
             features.append(
                 {
                     "type": "Feature",
-                        "geometry": geom,
+                    "geometry": json.loads(event.geometry),
                     "properties": {
-                            "distance_overtaker": event.distance_overtaker
+                        "distance_overtaker": event.distance_overtaker,
-                            if event.distance_overtaker is not None
+                        "distance_stationary": event.distance_stationary,
-                            and not math.isnan(event.distance_overtaker)
+                        "direction": -1 if event.direction_reversed else 1,
-                            else None,
-                            "distance_stationary": event.distance_stationary
-                            if event.distance_stationary is not None
-                            and not math.isnan(event.distance_stationary)
-                            else None,
-                            "direction": event.direction
-                            if event.direction is not None
-                            and not math.isnan(event.direction)
-                            else None,
                         "way_id": event.way_id,
-                            "course": event.course
+                        "course": event.course,
-                            if event.course is not None and not math.isnan(event.course)
+                        "speed": event.speed,
-                            else None,
+                        "time": event.time,
-                            "speed": event.speed
-                            if event.speed is not None and not math.isnan(event.speed)
-                            else None,
-                            "time": event.time_stamp,
-                            "zone": event.zone,
-                        },
-                    }
-                )
-
-            geojson = {"type": "FeatureCollection", "features": features}
-            return json_response(geojson)
-
-        raise InvalidUsage("unknown export format")
-
-
-@api.get(r"/export/segments")
-async def export_segments(req):
-    async with use_request_semaphore(req, "export_semaphore", timeout=30):
-        bbox = req.ctx.get_single_arg("bbox", default="-180,-90,180,90")
-        assert re.match(r"(-?\d+\.?\d+,?){4}", bbox)
-        bbox = list(map(float, bbox.split(",")))
-
-        fmt = req.ctx.get_single_arg("fmt", convert=ExportFormat)
-        segments = await req.ctx.db.stream(
-            text(
-                """
-                SELECT
-                    ST_AsGeoJSON(ST_Transform(geometry, 4326)) AS geometry,
-                    way_id,
-                    distance_overtaker_mean,
-                    distance_overtaker_min,
-                    distance_overtaker_max,
-                    distance_overtaker_median,
-                    overtaking_event_count,
-                    usage_count,
-                    direction,
-                    zone,
-                    offset_direction,
-                    distance_overtaker_array
-                FROM
-                    layer_obs_roads(
-                        ST_Transform(ST_MakeEnvelope(:bbox0, :bbox1, :bbox2, :bbox3, 4326), 3857),
-                        11,
-                        NULL,
-                        '1900-01-01'::timestamp,
-                        '2100-01-01'::timestamp
-                    )
-                WHERE usage_count > 0
-                """
-            ).bindparams(bbox0=bbox[0], bbox1=bbox[1], bbox2=bbox[2], bbox3=bbox[3])
-        )
-
-        if fmt == ExportFormat.SHAPEFILE:
-            with shapefile_zip(shape_type=3, basename="segments") as (
-                writer,
-                zip_buffer,
-            ):
-                writer.field("distance_overtaker_mean", "N", decimal=4)
-                writer.field("distance_overtaker_max", "N", decimal=4)
-                writer.field("distance_overtaker_min", "N", decimal=4)
-                writer.field("distance_overtaker_median", "N", decimal=4)
-                writer.field("overtaking_event_count", "N", decimal=4)
-                writer.field("usage_count", "N", decimal=4)
-                writer.field("way_id", "N", decimal=0)
-                writer.field("direction", "N", decimal=0)
-                writer.field("zone", "C")
-
-                async for segment in segments:
-                    geom = json.loads(segment.st_asgeojson)
-                    writer.line([geom["coordinates"]])
-                    writer.record(
-                        distance_overtaker_mean=segment.distance_overtaker_mean,
-                        distance_overtaker_median=segment.distance_overtaker_median,
-                        distance_overtaker_max=segment.distance_overtaker_max,
-                        distance_overtaker_min=segment.distance_overtaker_min,
-                        usage_count=segment.usage_count,
-                        overtaking_event_count=segment.overtaking_event_count,
-                        direction=segment.direction,
-                        way_id=segment.way_id,
-                        zone=segment.zone,
-                    )
-
-            return raw(zip_buffer.getbuffer())
-
-        if fmt == ExportFormat.GEOJSON:
-            features = []
-            async for segment in segments:
-                features.append(
-                    {
-                        "type": "Feature",
-                        "geometry": json.loads(segment.geometry),
-                        "properties": {
-                            "distance_overtaker_mean": segment.distance_overtaker_mean,
-                            "distance_overtaker_max": segment.distance_overtaker_max,
-                            "distance_overtaker_median": segment.distance_overtaker_median,
-                            "overtaking_event_count": segment.overtaking_event_count,
-                            "usage_count": segment.usage_count,
-                            "distance_overtaker_array": segment.distance_overtaker_array,
-                            "direction": segment.direction,
-                            "way_id": segment.way_id,
-                            "zone": segment.zone,
                     },
                 }
             )

api/obs/api/routes/frontend.py

@@ -14,7 +14,10 @@ if app.config.FRONTEND_CONFIG:
             **req.app.config.FRONTEND_CONFIG,
             "apiUrl": f"{req.ctx.api_url}/api",
             "loginUrl": f"{req.ctx.api_url}/login",
-            "obsMapSource": {
+            "obsMapSource": (
+                None
+                if app.config.LEAN_MODE
+                else {
                     "type": "vector",
                     "tiles": [
                         req.ctx.api_url
@@ -25,7 +28,8 @@ if app.config.FRONTEND_CONFIG:
                     ],
                     "minzoom": 0,
                     "maxzoom": 14,
-            },
+                }
+            ),
         }
 
         return response.json(result)

api/obs/api/routes/login.py

@@ -170,4 +170,4 @@ async def logout(req):
     auth_req = client.construct_EndSessionRequest(state=session["state"])
     logout_url = auth_req.request(client.end_session_endpoint)
 
-    return redirect(logout_url + f"&post_logout_redirect_uri={req.ctx.api_url}/logout")
+    return redirect(logout_url + f"&redirect_uri={req.ctx.api_url}/logout")

api/obs/api/routes/mapdetails.py

@@ -18,16 +18,14 @@ round_speed = partial(round_to, multiples=0.1)
 
 log = logging.getLogger(__name__)
 
-
+def get_bearing(a, b):
-def get_bearing(b, a):
     # longitude, latitude
     dL = b[0] - a[0]
     X = numpy.cos(b[1]) * numpy.sin(dL)
     Y = numpy.cos(a[1]) * numpy.sin(b[1]) - numpy.sin(a[1]) * numpy.cos(
         b[1]
     ) * numpy.cos(dL)
-    return numpy.arctan2(Y, X) + 0.5 * math.pi
+    return numpy.arctan2(X, Y)
-
 
 # Bins for histogram on overtaker distances. 0, 0.25, ... 2.25, infinity
 DISTANCE_BINS = numpy.arange(0, 2.5, 0.25).tolist() + [float('inf')]
@@ -84,11 +82,11 @@ async def mapdetails_road(req):
     arrays = numpy.array(arrays).T
 
     if len(arrays) == 0:
-        arrays = numpy.array([[], [], [], []], dtype=float)
+        arrays = numpy.array([[], [], [], []], dtype=numpy.float)
 
     data, mask = arrays[:-1], arrays[-1]
     data = data.astype(numpy.float64)
-    mask = mask.astype(bool)
+    mask = mask.astype(numpy.bool)
 
     def partition(arr, cond):
         return arr[:, cond], arr[:, ~cond]

api/obs/api/routes/stats.py

@@ -4,12 +4,12 @@ from typing import Optional
 from operator import and_
 from functools import reduce
 
-from sqlalchemy import distinct, select, func, desc
+from sqlalchemy import select, func, desc
 
 from sanic.response import json
 
 from obs.api.app import api
-from obs.api.db import Track, OvertakingEvent, User, Region, UserDevice
+from obs.api.db import Track, OvertakingEvent, User, Region
 from obs.api.utils import round_to
 
 
@@ -45,7 +45,7 @@ async def stats(req):
 
     # Only the user can look for their own stats, for now
     by_user = (
-        user is not None and req.ctx.user is not None and req.ctx.user.id == int(user)
+        user is not None and req.ctx.user is not None and req.ctx.user.username == user
     )
     if by_user:
         conditions.append(Track.author_id == req.ctx.user.id)
@@ -92,14 +92,6 @@ async def stats(req):
             .where(track_condition)
         )
     ).scalar()
-    device_count = (
-        await req.ctx.db.execute(
-            select(func.count(distinct(UserDevice.id)))
-            .select_from(UserDevice)
-            .join(Track.user_device)
-            .where(track_condition)
-        )
-    ).scalar()
 
     result = {
         "numEvents": event_count,
@@ -108,7 +100,6 @@ async def stats(req):
         "trackDuration": round_to(track_duration or 0, TRACK_DURATION_ROUNDING),
         "publicTrackCount": public_track_count,
         "trackCount": track_count,
-        "deviceCount": device_count,
     }
 
     return json(result)
@@ -183,7 +174,7 @@ async def stats(req):
     query = (
         select(
             [
-                Region.id,
+                Region.relation_id.label("id"),
                 Region.name,
                 func.count(OvertakingEvent.id).label("overtaking_event_count"),
             ]
@@ -191,11 +182,16 @@ async def stats(req):
         .select_from(Region)
         .join(
             OvertakingEvent,
-            func.ST_Within(OvertakingEvent.geometry, Region.geometry),
+            func.ST_Within(
+                func.ST_Transform(OvertakingEvent.geometry, 3857), Region.geometry
+            ),
         )
+        .where(Region.admin_level == 6)
         .group_by(
-            Region.id,
+            Region.relation_id,
             Region.name,
+            Region.relation_id,
+            Region.admin_level,
             Region.geometry,
         )
         .having(func.count(OvertakingEvent.id) > 0)

api/obs/api/routes/tiles.py

@@ -7,10 +7,10 @@ import dateutil.parser
 from sanic.exceptions import Forbidden, InvalidUsage
 from sanic.response import raw
 
-from sqlalchemy import text
+from sqlalchemy import select, text
+from sqlalchemy.sql.expression import table, column
 
 from obs.api.app import app
-from obs.api.utils import use_request_semaphore
 
 
 def get_tile(filename, zoom, x, y):
@@ -87,7 +87,6 @@ def get_filter_options(
 
 @app.route(r"/tiles/<zoom:int>/<x:int>/<y:(\d+)\.pbf>")
 async def tiles(req, zoom: int, x: int, y: str):
-    async with use_request_semaphore(req, "tile_semaphore"):
     if app.config.get("TILES_FILE"):
         tile = get_tile(req.app.config.TILES_FILE, int(zoom), int(x), int(y))
 
@@ -96,7 +95,7 @@ async def tiles(req, zoom: int, x: int, y: str):
 
         tile = await req.ctx.db.scalar(
             text(
-                    "select data from getmvt(:zoom, :x, :y, :user_id, :min_time, :max_time) as b(data, key);"
+                f"select data from getmvt(:zoom, :x, :y, :user_id, :min_time, :max_time) as b(data, key);"
             ).bindparams(
                 zoom=int(zoom),
                 x=int(x),

api/obs/api/routes/tracks.py

@@ -1,18 +1,16 @@
 import logging
 import re
-from datetime import date
 from json import load as jsonload
 from os.path import join, exists, isfile
 
-from sanic.exceptions import InvalidUsage, NotFound, Forbidden
+from sqlalchemy import select, func
-from sanic.response import file_stream, empty
-from slugify import slugify
-from sqlalchemy import select, func, and_
 from sqlalchemy.orm import joinedload
 
+from obs.api.db import Track, User, Comment, DuplicateTrackFileError
 from obs.api.app import api, require_auth, read_api_key, json
-from obs.api.db import Track, Comment, DuplicateTrackFileError
+
-from obs.api.utils import tar_of_tracks
+from sanic.response import file_stream, empty
+from sanic.exceptions import InvalidUsage, NotFound, Forbidden
 
 log = logging.getLogger(__name__)
 
@@ -25,8 +23,8 @@ def normalize_user_agent(user_agent):
     return m[0] if m else None
 
 
-async def _return_tracks(req, extend_query, limit, offset, order_by=None):
+async def _return_tracks(req, extend_query, limit, offset):
-    if limit <= 0 or limit > 1000:
+    if limit <= 0 or limit > 100:
         raise InvalidUsage("invalid limit")
 
     if offset < 0:
@@ -41,7 +39,7 @@ async def _return_tracks(req, extend_query, limit, offset, order_by=None):
         extend_query(select(Track).options(joinedload(Track.author)))
         .limit(limit)
         .offset(offset)
-        .order_by(order_by if order_by is not None else Track.created_at)
+        .order_by(Track.created_at.desc())
     )
 
     tracks = (await req.ctx.db.execute(query)).scalars()
@@ -78,101 +76,16 @@ async def get_tracks(req):
     return await _return_tracks(req, extend_query, limit, offset)
 
 
-def parse_boolean(s):
-    if s is None:
-        return None
-
-    s = s.lower()
-    if s in ("true", "1", "yes", "y", "t"):
-        return True
-    if s in ("false", "0", "no", "n", "f"):
-        return False
-
-    raise ValueError("invalid value for boolean")
-
-
 @api.get("/tracks/feed")
 @require_auth
 async def get_feed(req):
     limit = req.ctx.get_single_arg("limit", default=20, convert=int)
     offset = req.ctx.get_single_arg("offset", default=0, convert=int)
-    user_device_id = req.ctx.get_single_arg("user_device_id", default=None, convert=int)
-
-    order_by_columns = {
-        "recordedAt": Track.recorded_at,
-        "title": Track.title,
-        "visibility": Track.public,
-        "length": Track.length,
-        "duration": Track.duration,
-        "user_device_id": Track.user_device_id,
-    }
-    order_by = req.ctx.get_single_arg(
-        "order_by", default=None, convert=order_by_columns.get
-    )
-
-    reversed_ = req.ctx.get_single_arg("reversed", convert=parse_boolean, default=False)
-    if reversed_:
-        order_by = order_by.desc()
-
-    public = req.ctx.get_single_arg("public", convert=parse_boolean, default=None)
 
     def extend_query(q):
-        q = q.where(Track.author_id == req.ctx.user.id)
+        return q.where(Track.author_id == req.ctx.user.id)
 
-        if user_device_id is not None:
+    return await _return_tracks(req, extend_query, limit, offset)
-            q = q.where(Track.user_device_id == user_device_id)
-
-        if public is not None:
-            q = q.where(Track.public == public)
-
-        return q
-
-    return await _return_tracks(req, extend_query, limit, offset, order_by)
-
-
-@api.post("/tracks/bulk")
-@require_auth
-async def tracks_bulk_action(req):
-    body = req.json
-    action = body["action"]
-    track_slugs = body["tracks"]
-
-    if action not in ("delete", "makePublic", "makePrivate", "reprocess", "download"):
-        raise InvalidUsage("invalid action")
-
-    query = select(Track).where(
-        and_(Track.author_id == req.ctx.user.id, Track.slug.in_(track_slugs))
-    )
-
-    files = set()
-
-    for track in (await req.ctx.db.execute(query)).scalars():
-        if action == "delete":
-            await req.ctx.db.delete(track)
-        elif action == "makePublic":
-            if not track.public:
-                track.queue_processing()
-            track.public = True
-        elif action == "makePrivate":
-            if track.public:
-                track.queue_processing()
-            track.public = False
-        elif action == "reprocess":
-            track.queue_processing()
-        elif action == "download":
-            files.add(track.get_original_file_path(req.app.config))
-
-    await req.ctx.db.commit()
-
-    if action == "download":
-        username_slug = slugify(req.ctx.user.username, separator="-")
-        date_str = date.today().isoformat()
-        file_basename = f"tracks_{username_slug}_{date_str}"
-
-        await tar_of_tracks(req, files, file_basename)
-        return
-
-    return empty()
 
 
 @api.post("/tracks")

api/obs/api/routes/users.py

@@ -1,11 +1,9 @@
 import logging
 
 from sanic.response import json
-from sanic.exceptions import InvalidUsage, Forbidden, NotFound
+from sanic.exceptions import InvalidUsage
-from sqlalchemy import and_, select
 
 from obs.api.app import api, require_auth
-from obs.api.db import UserDevice
 
 log = logging.getLogger(__name__)
 
@@ -30,48 +28,6 @@ async def get_user(req):
     return json(user_to_json(req.ctx.user) if req.ctx.user else None)
 
 
-@api.get("/user/devices")
-async def get_user_devices(req):
-    if not req.ctx.user:
-        raise Forbidden()
-
-    query = (
-        select(UserDevice)
-        .where(UserDevice.user_id == req.ctx.user.id)
-        .order_by(UserDevice.id)
-    )
-
-    devices = (await req.ctx.db.execute(query)).scalars()
-
-    return json([device.to_dict(req.ctx.user.id) for device in devices])
-
-
-@api.put("/user/devices/<device_id:int>")
-async def put_user_device(req, device_id):
-    if not req.ctx.user:
-        raise Forbidden()
-
-    body = req.json
-
-    query = (
-        select(UserDevice)
-        .where(and_(UserDevice.user_id == req.ctx.user.id, UserDevice.id == device_id))
-        .limit(1)
-    )
-
-    device = (await req.ctx.db.execute(query)).scalar()
-
-    if device is None:
-        raise NotFound()
-
-    new_name = body.get("displayName", "").strip()
-    if new_name and device.display_name != new_name:
-        device.display_name = new_name
-        await req.ctx.db.commit()
-
-    return json(device.to_dict())
-
-
 @api.put("/user")
 @require_auth
 async def put_user(req):

api/obs/api/utils.py

@@ -1,15 +1,6 @@
-import asyncio
-from contextlib import asynccontextmanager
 from datetime import datetime
-import logging
-from os.path import commonpath, join, relpath
-import queue
-import tarfile
-
 import dateutil.parser
-from sanic.exceptions import InvalidUsage, ServiceUnavailable
+from sanic.exceptions import InvalidUsage
-
-log = logging.getLogger(__name__)
 
 RAISE = object()
 
@@ -39,124 +30,3 @@ def round_to(value: float, multiples: float) -> float:
     if value is None:
         return None
     return round(value / multiples) * multiples
-
-
-def chunk_list(lst, n):
-    for s in range(0, len(lst), n):
-        yield lst[s : s + n]
-
-
-class chunk:
-    def __init__(self, iterable, n):
-        self.iterable = iterable
-        self.n = n
-
-    def __iter__(self):
-        if isinstance(self.iterable, list):
-            yield from chunk_list(self.iterable, self.n)
-            return
-
-        it = iter(self.iterable)
-        while True:
-            current = []
-            try:
-                for _ in range(self.n):
-                    current.append(next(it))
-                yield current
-            except StopIteration:
-                if current:
-                    yield current
-                break
-
-    async def __aiter__(self):
-        if hasattr(self.iterable, "__iter__"):
-            for item in self:
-                yield item
-            return
-
-        it = self.iterable.__aiter__()
-        while True:
-            current = []
-            try:
-                for _ in range(self.n):
-                    current.append(await it.__anext__())
-                yield current
-            except StopAsyncIteration:
-                if len(current):
-                    yield current
-                break
-
-
-async def tar_of_tracks(req, files, file_basename="tracks"):
-    response = await req.respond(
-        content_type="application/x-gtar",
-        headers={
-            "content-disposition": f'attachment; filename="{file_basename}.tar.bz2"'
-        },
-    )
-
-    helper = StreamerHelper(response)
-
-    tar = tarfile.open(name=None, fileobj=helper, mode="w|bz2", bufsize=256 * 512)
-
-    root = commonpath(list(files))
-    for fname in files:
-        log.info("Write file to tar: %s", fname)
-        with open(fname, "rb") as fobj:
-            tarinfo = tar.gettarinfo(fname)
-            tarinfo.name = join(file_basename, relpath(fname, root))
-            tar.addfile(tarinfo, fobj)
-            await helper.send_all()
-    tar.close()
-    await helper.send_all()
-
-    await response.eof()
-
-
-class StreamerHelper:
-    def __init__(self, response):
-        self.response = response
-        self.towrite = queue.Queue()
-
-    def write(self, data):
-        self.towrite.put(data)
-
-    async def send_all(self):
-        while True:
-            try:
-                tosend = self.towrite.get(block=False)
-                await self.response.send(tosend)
-            except queue.Empty:
-                break
-
-
-@asynccontextmanager
-async def use_request_semaphore(req, semaphore_name, timeout=10):
-    """
-    If configured, acquire a semaphore for the map tile request and release it
-    after the context has finished.
-
-    If the semaphore cannot be acquired within the timeout, issue a 503 Service
-    Unavailable error response that describes that the database is overloaded,
-    so users know what the problem is.
-
-    Operates as a noop when the tile semaphore is not enabled.
-    """
-    semaphore = getattr(req.app.ctx, semaphore_name, None)
-
-    if semaphore is None:
-        yield
-        return
-
-    try:
-        await asyncio.wait_for(semaphore.acquire(), timeout)
-
-        try:
-            yield
-        finally:
-            semaphore.release()
-
-    except asyncio.TimeoutError:
-        raise ServiceUnavailable(
-            "Too many requests, database overloaded. Please retry later."
-        )

api/obs/bin/openbikesensor_api.py

@@ -58,7 +58,7 @@ def main():
         port=app.config.PORT,
         debug=debug,
         auto_reload=app.config.get("AUTO_RELOAD", debug),
-        access_log=True,
+        # access_log=False,
     )
 
 

api/obs/bin/openbikesensor_transform_osm.py

@@ -1,191 +0,0 @@
-#!/usr/bin/env python3
-
-import sys
-import re
-import msgpack
-
-import osmium
-import shapely.wkb as wkb
-from shapely.ops import transform
-
-HIGHWAY_TYPES = {
-    "trunk",
-    "primary",
-    "secondary",
-    "tertiary",
-    "unclassified",
-    "residential",
-    "trunk_link",
-    "primary_link",
-    "secondary_link",
-    "tertiary_link",
-    "living_street",
-    "service",
-    "track",
-    "road",
-}
-ZONE_TYPES = {
-    "urban",
-    "rural",
-    "motorway",
-}
-URBAN_TYPES = {
-    "residential",
-    "living_street",
-    "road",
-}
-MOTORWAY_TYPES = {
-    "motorway",
-    "motorway_link",
-}
-
-ADMIN_LEVEL_MIN = 2
-ADMIN_LEVEL_MAX = 8
-MINSPEED_RURAL = 60
-
-ONEWAY_YES = {"yes", "true", "1"}
-ONEWAY_REVERSE = {"reverse", "-1"}
-
-
-def parse_number(tag):
-    if not tag:
-        return None
-
-    match = re.search(r"[0-9]+", tag)
-    if not match:
-        return None
-
-    digits = match.group(0)
-    try:
-        return int(digits)
-    except ValueError:
-        return None
-
-
-def determine_zone(tags):
-    highway = tags.get("highway")
-    zone = tags.get("zone:traffic")
-
-    if zone is not None:
-        if "rural" in zone:
-            return "rural"
-
-        if "motorway" in zone:
-            return "motorway"
-
-        return "urban"
-
-    # From here on we are guessing based on other tags
-
-    if highway in URBAN_TYPES:
-        return "urban"
-
-    if highway in MOTORWAY_TYPES:
-        return "motorway"
-
-    maxspeed_source = tags.get("source:maxspeed")
-    if maxspeed_source and "rural" in maxspeed_source:
-        return "rural"
-    if maxspeed_source and "urban" in maxspeed_source:
-        return "urban"
-
-    for key in ["maxspeed", "maxspeed:forward", "maxspeed:backward"]:
-        maxspeed = parse_number(tags.get(key))
-        if maxspeed is not None and maxspeed > MINSPEED_RURAL:
-            return "rural"
-
-    # default to urban if we have no idea
-    return "urban"
-
-
-def determine_direction(tags, zone):
-    if (
-        tags.get("oneway") in ONEWAY_YES
-        or tags.get("junction") == "roundabout"
-        or zone == "motorway"
-    ):
-        return 1, True
-
-    if tags.get("oneway") in ONEWAY_REVERSE:
-        return -1, True
-
-    return 0, False
-
-
-class StreamPacker:
-    def __init__(self, stream, *args, **kwargs):
-        self.stream = stream
-        self.packer = msgpack.Packer(*args, autoreset=False, **kwargs)
-
-    def _write_out(self):
-        if hasattr(self.packer, "getbuffer"):
-            chunk = self.packer.getbuffer()
-        else:
-            chunk = self.packer.bytes()
-
-        self.stream.write(chunk)
-        self.packer.reset()
-
-    def pack(self, *args, **kwargs):
-        self.packer.pack(*args, **kwargs)
-        self._write_out()
-
-    def pack_array_header(self, *args, **kwargs):
-        self.packer.pack_array_header(*args, **kwargs)
-        self._write_out()
-
-    def pack_map_header(self, *args, **kwargs):
-        self.packer.pack_map_header(*args, **kwargs)
-        self._write_out()
-
-    def pack_map_pairs(self, *args, **kwargs):
-        self.packer.pack_map_pairs(*args, **kwargs)
-        self._write_out()
-
-
-# A global factory that creates WKB from a osmium geometry
-wkbfab = osmium.geom.WKBFactory()
-
-from pyproj import Transformer
-
-project = Transformer.from_crs("EPSG:4326", "EPSG:3857", always_xy=True).transform
-
-
-class OSMHandler(osmium.SimpleHandler):
-    def __init__(self, packer):
-        self.packer = packer
-        super().__init__()
-
-    def way(self, way):
-        tags = way.tags
-
-        highway = tags.get("highway")
-        if not highway or highway not in HIGHWAY_TYPES:
-            return
-
-        access = tags.get("access", None)
-        bicycle = tags.get("bicycle", None)
-        if access == "no" and bicycle not in ["designated", "yes", "permissive", "destination"]:
-            return
-
-        zone = determine_zone(tags)
-        directionality, oneway = determine_direction(tags, zone)
-        name = tags.get("name")
-
-        geometry = wkb.loads(wkbfab.create_linestring(way), hex=True)
-        geometry = transform(project, geometry)
-        geometry = wkb.dumps(geometry)
-        self.packer.pack(
-            [b"\x01", way.id, name, zone, directionality, oneway, geometry]
-        )
-
-
-def main():
-    with open(sys.argv[2], "wb") as fout:
-        packer = StreamPacker(fout)
-        osmhandler = OSMHandler(packer)
-        osmhandler.apply_file(sys.argv[1], locations=True)
-
-
-if __name__ == "__main__":
-    main()

api/requirements.txt

@@ -1,22 +1,14 @@
 coloredlogs~=15.0.1
-sanic==22.6.2
+sanic~=22.6.0
-oic~=1.5.0
+oic~=1.3.0
 sanic-session~=0.8.0
+sanic-cors~=2.0.1
 python-slugify~=6.1.2
-motor~=3.1.1
+motor~=3.0.0
-pyyaml~=5.3.1
+pyyaml<6
 -e git+https://github.com/openmaptiles/openmaptiles-tools#egg=openmaptiles-tools
-sqlparse~=0.4.3
+sqlparse~=0.4.2
-sqlalchemy[asyncio]~=1.4.46
+sqlalchemy[asyncio]~=1.4.39
-asyncpg~=0.27.0
+asyncpg~=0.24.0
 pyshp~=2.3.1
-alembic~=1.9.4
+alembic~=1.7.7
-stream-zip~=0.0.50
-msgpack~=1.0.5
-osmium~=3.6.0
-psycopg~=3.1.8
-shapely~=2.0.1
-pyproj~=3.4.1
-aiohttp~=3.8.1
-# sanic requires websocets and chockes on >=10 in 2022.6.2
-websockets<11

api/scripts

@@ -1 +1 @@
-Subproject commit 664e4d606416417c0651ea1748d32dd36209be6a
+Subproject commit 8e9395fd3cd0f1e83b4413546bc2d3cb0c726738

api/setup.py

@@ -11,24 +11,23 @@ setup(
     package_data={},
     install_requires=[
         "coloredlogs~=15.0.1",
-        "sanic==22.6.2",
+        "sanic>=21.9.3,<22.7.0",
         "oic>=1.3.0, <2",
         "sanic-session~=0.8.0",
+        "sanic-cors~=2.0.1",
         "python-slugify>=5.0.2,<6.2.0",
-        "motor>=2.5.1,<3.1.2",
+        "motor>=2.5.1,<3.1.0",
         "pyyaml<6",
-        "sqlparse~=0.4.3",
+        "sqlparse~=0.4.2",
         "openmaptiles-tools",  # install from git
         "pyshp>=2.2,<2.4",
-        "sqlalchemy[asyncio]~=1.4.46",
+        "sqlalchemy[asyncio]~=1.4.25",
-        "asyncpg~=0.27.0",
+        "asyncpg~=0.24.0",
-        "alembic~=1.9.4",
+        "alembic~=1.7.7",
-        "stream-zip~=0.0.50",
     ],
     entry_points={
         "console_scripts": [
             "openbikesensor-api=obs.bin.openbikesensor_api:main",
-            "openbikesensor-transform-osm=obs.bin.openbikesensor_transform_osm:main",
         ]
     },
 )

api/tools/import_osm.py

@@ -1,108 +0,0 @@
-#!/usr/bin/env python3
-
-from dataclasses import dataclass
-import asyncio
-from os.path import basename, splitext
-import sys
-import logging
-
-import msgpack
-import psycopg
-
-from obs.api.app import app
-from obs.api.utils import chunk
-
-log = logging.getLogger(__name__)
-
-
-ROAD_BUFFER = 1000
-AREA_BUFFER = 100
-
-
-@dataclass
-class Road:
-    way_id: int
-    name: str
-    zone: str
-    directionality: int
-    oneway: int
-    geometry: bytes
-
-
-def read_file(filename):
-    """
-    Reads a file iteratively, yielding
-    appear. Those may be mixed.
-    """
-
-    with open(filename, "rb") as f:
-        unpacker = msgpack.Unpacker(f)
-        try:
-            while True:
-                type_id, *data = unpacker.unpack()
-
-                if type_id == b"\x01":
-                    yield Road(*data)
-
-        except msgpack.OutOfData:
-            pass
-
-
-async def import_osm(connection, filename, import_group=None):
-    if import_group is None:
-        import_group = splitext(basename(filename))[0]
-
-    # Pass 1: Find IDs only
-    road_ids = []
-    for item in read_file(filename):
-        road_ids.append(item.way_id)
-
-    async with connection.cursor() as cursor:
-        log.info("Pass 1: Delete previously imported data")
-
-        log.debug("Delete import group %s", import_group)
-        await cursor.execute(
-            "DELETE FROM road WHERE import_group = %s", (import_group,)
-        )
-
-        log.debug("Delete roads by way_id")
-        for ids in chunk(road_ids, 10000):
-            await cursor.execute("DELETE FROM road WHERE way_id = ANY(%s)", (ids,))
-
-        # Pass 2: Import
-        log.info("Pass 2: Import roads")
-        amount = 0
-        for items in chunk(read_file(filename), 10000):
-            amount += 10000
-            log.info(f"...{amount}/{len(road_ids)} ({100*amount/len(road_ids)}%)")
-            async with cursor.copy(
-                "COPY road (way_id, name, zone, directionality, oneway, geometry, import_group) FROM STDIN"
-            ) as copy:
-                for item in items:
-                    await copy.write_row(
-                        (
-                            item.way_id,
-                            item.name,
-                            item.zone,
-                            item.directionality,
-                            item.oneway,
-                            bytes.hex(item.geometry),
-                            import_group,
-                        )
-                    )
-
-
-async def main():
-    logging.basicConfig(level=logging.DEBUG, format="%(levelname)s: %(message)s")
-
-    url = app.config.POSTGRES_URL
-    url = url.replace("+asyncpg", "")
-
-    async with await psycopg.AsyncConnection.connect(url) as connection:
-        for filename in sys.argv[1:]:
-            log.debug("Loading file: %s", filename)
-            await import_osm(connection, filename)
-
-
-if __name__ == "__main__":
-    asyncio.run(main())

api/tools/import_regions.py

@@ -1,93 +0,0 @@
-#!/usr/bin/env python3
-
-"""
-This script downloads and/or imports regions for statistical analysis into the
-PostGIS database. The regions are sourced from:
-
-* EU countries are covered by
-  [NUTS](https://ec.europa.eu/eurostat/web/gisco/geodata/reference-data/administrative-units-statistical-units/nuts).
-"""
-
-import tempfile
-from dataclasses import dataclass
-import json
-import asyncio
-from os.path import basename, splitext
-import sys
-import logging
-from typing import Optional
-
-import aiohttp
-import psycopg
-
-from obs.api.app import app
-from obs.api.utils import chunk
-
-log = logging.getLogger(__name__)
-
-NUTS_URL = "https://gisco-services.ec.europa.eu/distribution/v2/nuts/geojson/NUTS_RG_01M_2021_3857.geojson"
-
-from pyproj import Transformer
-
-project = Transformer.from_crs("EPSG:4326", "EPSG:3857", always_xy=True).transform
-from shapely.ops import transform
-from shapely.geometry import shape
-import shapely.wkb as wkb
-
-
-async def import_nuts(
-    connection, filename=None, level: int = 3, import_group: Optional[str] = None
-):
-    if import_group is None:
-        import_group = f"nuts{level}"
-
-    if filename:
-        log.info("Load NUTS from file")
-        with open(filename) as f:
-            data = json.load(f)
-    else:
-        log.info("Download NUTS regions from europa.eu")
-        async with aiohttp.ClientSession() as session:
-            async with session.get(NUTS_URL) as resp:
-                data = await resp.json(content_type=None)
-
-    async with connection.cursor() as cursor:
-        log.info(
-            "Delete previously imported regions with import group %s", import_group
-        )
-        await cursor.execute(
-            "DELETE FROM region WHERE import_group = %s", (import_group,)
-        )
-
-        log.info("Import regions")
-        async with cursor.copy(
-            "COPY region (id, name, geometry, import_group) FROM STDIN"
-        ) as copy:
-            for feature in data["features"]:
-                if feature["properties"]["LEVL_CODE"] == level:
-                    geometry = shape(feature["geometry"])
-                    # geometry = transform(project, geometry)
-                    geometry = wkb.dumps(geometry)
-                    geometry = bytes.hex(geometry)
-                    await copy.write_row(
-                        (
-                            feature["properties"]["NUTS_ID"],
-                            feature["properties"]["NUTS_NAME"],
-                            geometry,
-                            import_group,
-                        )
-                    )
-
-
-async def main():
-    logging.basicConfig(level=logging.DEBUG, format="%(levelname)s: %(message)s")
-
-    url = app.config.POSTGRES_URL
-    url = url.replace("+asyncpg", "")
-
-    async with await psycopg.AsyncConnection.connect(url) as connection:
-        await import_nuts(connection, sys.argv[1] if len(sys.argv) > 1 else None)
-
-
-if __name__ == "__main__":
-    asyncio.run(main())

api/tools/prepare_sql_tiles.py

@@ -32,6 +32,32 @@ EXTRA_ARGS = [
 ]
 
 
+class CustomMvtGenerator(MvtGenerator):
+    def generate_sqltomvt_func(self, fname, extra_args: List[Tuple[str, str]]) -> str:
+        """
+        Creates a SQL function that returns a single bytea value or null. This
+        method is overridden to allow for custom arguments in the created function
+        """
+        extra_args_types = "".join([f", {a[1]}" for a in extra_args])
+        extra_args_definitions = "".join(
+            [f", {a[0]} {a[1]} DEFAULT {a[2]}" for a in extra_args]
+        )
+
+        return f"""\
+DROP FUNCTION IF EXISTS {fname}(integer, integer, integer{extra_args_types});
+CREATE FUNCTION {fname}(zoom integer, x integer, y integer{extra_args_definitions})
+RETURNS {'TABLE(mvt bytea, key text)' if self.key_column else 'bytea'} AS $$
+{self.generate_sql()};
+$$ LANGUAGE SQL STABLE CALLED ON NULL INPUT;"""
+
+EXTRA_ARGS = [
+    # name, type, default
+    ("user_id", "integer", "NULL"),
+    ("min_time", "timestamp", "NULL"),
+    ("max_time", "timestamp", "NULL"),
+]
+
+
 class CustomMvtGenerator(MvtGenerator):
     def generate_sqltomvt_func(self, fname, extra_args: List[Tuple[str, str]]) -> str:
         """

api/tools/reimport_tracks.py

@@ -1,30 +0,0 @@
-#!/usr/bin/env python3
-import logging
-import asyncio
-
-from sqlalchemy import text
-
-from obs.api.app import app
-from obs.api.db import connect_db, make_session
-
-log = logging.getLogger(__name__)
-
-async def main():
-    logging.basicConfig(level=logging.DEBUG, format="%(levelname)s: %(message)s")
-    await reimport_tracks()
-
-
-async def reimport_tracks():
-
-    async with connect_db(
-        app.config.POSTGRES_URL,
-        app.config.POSTGRES_POOL_SIZE,
-        app.config.POSTGRES_MAX_OVERFLOW,
-    ):
-        async with make_session() as session:
-            await session.execute(text("UPDATE track SET processing_status = 'queued';"))
-            await session.commit()
-
-
-if __name__ == "__main__":
-    asyncio.run(main())

api/tools/transform_osm.py

@@ -1,6 +0,0 @@
-#!/usr/bin/env python3
-
-from obs.bin.openbikesensor_transform_osm import main
-
-if __name__ == "__main__":
-    main()

api/tools/upgrade.py

@@ -1,15 +1,14 @@
 #!/usr/bin/env python3
-import asyncio
 import logging
+import asyncio
+from alembic.config import Config
+from alembic import command
+from os.path import join, dirname
 
 log = logging.getLogger(__name__)
 
 from prepare_sql_tiles import prepare_sql_tiles, _run
 
-from import_regions import main as import_nuts
-
-from reimport_tracks import main as reimport_tracks
-
 
 async def _migrate():
     await _run("alembic upgrade head")
@@ -21,11 +20,7 @@ async def main():
     await _migrate()
     log.info("Preparing SQL tiles...")
     await prepare_sql_tiles()
-    log.info("Importing nuts regions...")
+    log.info("Upgraded")
-    await import_nuts()
-    log.info("Nuts regions imported, scheduling reimport of tracks")
-    await reimport_tracks()
-
 
 
 if __name__ == "__main__":

deployment/README.md

@@ -55,7 +55,12 @@ git clone --recursive https://github.com/openbikesensor/portal source/
 ```bash
 mkdir -p /opt/openbikesensor/config
 cd /opt/openbikesensor/
-cp -r source/deployment/config source/deployment/docker-compose.yaml source/deployment/.env .
+
+cp source/deployment/examples/docker-compose.yaml docker-compose.yaml
+cp source/deployment/examples/.env .env
+
+cp source/deployment/examples/traefik.toml config/traefik.toml
+cp source/deployment/examples/config.py config/config.py
 ```
 
 ### Create a Docker network
@@ -219,6 +224,18 @@ docker-compose build portal
 
 *Hint*: This may take up to 10 minutes. In the future, we will provide a prebuild image.
 
+#### Download OpenStreetMap maps
+
+Download the area(s) you would like to import from 
+[GeoFabrik](https://download.geofabrik.de) into `data/pbf`, for example:
+
+```bash
+cd /opt/openbikesensor/
+wget https://download.geofabrik.de/europe/germany/schleswig-holstein-latest.osm.pbf -P data/pbf
+```
+
+*Hint*: Start with a small region/city, since the import can take some hours for huge areas.
+
 #### Prepare database
 
 Run the following scripts to prepare the database:
@@ -231,7 +248,13 @@ For more details, see [README.md](../README.md) under "Prepare database".
 
 #### Import OpenStreetMap data
 
-Follow [these instructions](./osm-import.md).
+Run the following script, to import the OSM data:
+
+```
+docker-compose run --rm portal tools/osm2pgsql.sh
+```
+
+For more details. see [README.md](../README.md) under "Import OpenStreetMap data".
 
 
 #### Configure portal
@@ -297,7 +320,7 @@ You should see smth. like:
 
 When you click on *My Tracks*, you should see it on a map.
 
-#### Configure the map position
+#### Configre the map position
 
 Open the tab *Map** an zoom to the desired position. The URL contains the corresponding GPS position,
 for example:
@@ -318,6 +341,10 @@ docker-compose restart portal
 The tab *Map* should be the selected map section now.
 When you uploaded some tracks, you map should show a colors overlay on the streets.
 
+#### Verify osm2pgsql
+
+If you zoom in the tab *Map* at the imported region/city, you should see dark grey lines on the streets.
+
 ## Miscellaneous
 
 ### Logs

deployment/config/nginx.conf

@@ -1,22 +0,0 @@
-
-events {}
-http {
-    proxy_cache_path  /data/nginx/cache  levels=1:2    keys_zone=STATIC:10m
-    inactive=24h  max_size=1g;
-    server {
-        location ~* ^/tiles/\d[012]?/[^?]+$ {
-            proxy_pass             http://portal:3000;
-            proxy_set_header       Host $host:3000;
-            proxy_buffering        on;
-            proxy_cache_methods    GET HEAD;
-            proxy_cache            STATIC;
-            proxy_cache_valid      200  1d;
-            proxy_cache_use_stale  error timeout invalid_header updating
-                                   http_500 http_502 http_503 http_504;
-        }
-        location / {
-            proxy_pass             http://portal:3000;
-            proxy_set_header       Host $host:3000;
-        }
-    }
-}

deployment/examples/config.py

@@ -5,7 +5,12 @@
 # Extended log output, but slower
 DEBUG = False
 VERBOSE = DEBUG
-AUTO_RELOAD = DEBUG
+AUTO_RESTART = DEBUG
+
+# Turn on lean mode to simplify the setup. Lots of features will be
+# unavailable, but you will not need to manage OpenStreetMap data. Please make
+# sure to configure the OBS_FACE_CACHE_DIR correctly for lean mode.
+LEAN_MODE = False
 
 # Required to encrypt or sign sessions, cookies, tokens, etc.
 #SECRET = "!!!<<<CHANGEME>>>!!!"

deployment/examples/docker-compose.yaml

@@ -14,7 +14,7 @@ services:
   ############################################################
 
   postgres:
-    image: "openmaptiles/postgis:7.0"
+    image: "openmaptiles/postgis:6.0"
     environment:
       - POSTGRES_DB=${OBS_POSTGRES_DB}
       - POSTGRES_USER=${OBS_POSTGRES_USER}
@@ -136,7 +136,7 @@ services:
       - "traefik.docker.network=gateway"
 
   postgres-keycloak:
-    image: postgres:15
+    image: postgres:13.3
     restart: always
     networks:
       - backend

docker-compose.yaml

@@ -8,7 +8,7 @@ version: '3'
 
 services:
   postgres:
-    image: "openmaptiles/postgis:7.0"
+    image: "openmaptiles/postgis:6.0"
     environment:
       POSTGRES_USER: obs
       POSTGRES_PASSWORD: obs
@@ -20,7 +20,6 @@ services:
 
   api:
     image: openbikesensor-api
-    tty: true
     build:
       context: ./api/
       dockerfile: Dockerfile
@@ -36,8 +35,6 @@ services:
       - ./tile-generator/data/:/tiles
       - ./api/migrations:/opt/obs/api/migrations
       - ./api/alembic.ini:/opt/obs/api/alembic.ini
-      - ./local/pbf:/pbf
-      - ./local/obsdata:/obsdata
     depends_on:
       - postgres
       - keycloak
@@ -49,7 +46,6 @@ services:
 
   worker:
     image: openbikesensor-api
-    tty: true
     build:
       context: ./api/
       dockerfile: Dockerfile

docs/lean-mode.md

@@ -0,0 +1,57 @@
+# Lean mode
+
+The application can be configured in "lean mode" through the `LEAN_MODE`
+setting in `config.py`. A lean installation is easier to set up, as a few steps
+can be skipped. However, the performance of the application will degrade in
+lean mode, and lots of advanced features will not be available.
+
+Lean mode is meant as an entrypoint to get started with collecting data,
+without the hassle of importing and maintaining OpenStreetMap data.
+
+## Disabled features in lean mode
+
+* No map tiles are generated.
+* The frontend will not show an overview map, only per-track maps.
+* The `roads` database table is not used, neither for processing tracks, nor
+  for generating map tiles.
+* The API will not generate auxiliary information for display on the
+  (nonexistent) map, such as per-road statistics.
+
+## Switch to/from lean mode 
+
+To enable lean mode, set the following in your `config.py` (or in
+`config.overrides.py`, especially in development setups):
+
+```python
+LEAN_MODE = True
+```
+
+To disable lean mode, set it to `False` instead.
+
+For lean mode, it is important that the config variable `OBS_FACE_CACHE_DIR` is
+properly set, or that you are happy with its default value of using
+`$DATA_DIR/obs-face-cache`.
+
+When turning off lean mode, make sure to fill your `roads` table properly, as
+otherwise the track processing will not work. When turning on lean mode, you
+may truncate the `roads` table to save space, but you don't need to, it simply
+becomes unused.
+
+## Benefits
+
+* When using lean mode, you can skip the import of OpenStreetMap data during
+  setup, and you also do not need to keep it updated. 
+* People can already start uploading data and the data is also processed,
+  giving you as a maintainer more time to set up the full application, if you
+  want to.
+
+## Drawbacks
+
+* Lean mode is less performant when processing tracks.
+* Lean mode track processing depends on the Overpass API data source, which may
+  be slow, unavailable, or rate limiting the requests, so processing may fail.
+  We use caching to prevent some issues, but as we depend on a third party
+  service here that is accessed for free and that generates a lot of server
+  load, we really can't ask for much. If you frequently run into issues, the
+  best bet is to manage OSM data yourself and turn off lean mode.
+* Of course some features are missing.

docs/osm-import.md

@@ -1,103 +0,0 @@
-# Importing OpenStreetMap data
-
-The application requires a lot of data from the OpenStreetMap to work.
-
-The required information is stored in the PostgreSQL database and used when
-processing tracks, as well as for vector tile generation. The process applies
-to both development and production setups. For development, you should choose a
-small area for testing, such as your local county or city, to keep the amount
-of data small. For production use you have to import the whole region you are
-serving.
-
-## General pipeline overview
-
-1. Download OpenStreetMap data as one or more `.osm.pbf` files.
-2. Transform this data to generate geometry data for all roads and regions, so
-   we don't need to look up nodes separately. This step requires a lot of CPU
-   and memory, so it can be done "offline" on a high power machine.
-3. Import the transformed data into the PostgreSQL/PostGIS database.
-
-## Community hosted transformed data
-
-Since the first two steps are the same for everybody, the community will soon
-provide a service where relatively up-to-date transformed data can be
-downloaded for direct import. Stay tuned.
-
-## Download data
-
-[GeoFabrik](https://download.geofabrik.de) kindly hosts extracts of the
-OpenStreetMap planet by region. Download all regions you're interested in from
-there in `.osm.pbf` format, with the tool of your choice, e. g.:
-
-```bash
-wget -P local/pbf/ https://download.geofabrik.de/europe/germany/baden-wuerttemberg-latest.osm.pbf 
-```
-
-## Transform data
-
-To transform downloaded data, you can either use the docker image from a
-development or production environment, or locally install the API into your
-python environment. Then run the `api/tools/transform_osm.py` script on the data.
-
-```bash
-api/tools/transform_osm.py baden-wuerttemberg-latest.osm.pbf baden-wuerttemberg-latest.msgpack
-```
-
-In dockerized setups, make sure to mount your data somewhere in the container
-and also mount a directory where the result can be written. The development
-setup takes care of this, so you can use:
-
-```bash
-docker-compose run --rm api tools/transform_osm.py \
-  /pbf/baden-wuerttemberg-latest.osm.pbf /obsdata/baden-wuerttemberg-latest.msgpack
-```
-
-Repeat this command for every file you want to transform.
-
-## Import transformed data
-
-The command for importing looks like this:
-
-```bash
-api/tools/import_osm.py baden-wuerttemberg-latest.msgpack
-```
-
-This tool reads your application config from `config.py`, so set that up first
-as if you were setting up your application.
-
-In dockerized setups, make sure to mount your data somewhere in the container.
-Again, the development setup takes care of this, so you can use:
-
-```bash
-docker-compose run --rm api tools/import_osm.py \
-  /obsdata/baden-wuerttemberg-latest.msgpack
-```
-
-The transform process should take a few seconds to minutes, depending on the area
-size. You can run the process multiple times, with the same or different area
-files, to import or update the data. You can update only one region and leave
-the others as they are, or add more filenames to the command line to
-bulk-import data.
-
-## How this works
-
-* The transformation is done with a python script that uses
-  [pyosmium](https://osmcode.org/pyosmium/) to read the `.osm.pbf` file. This
-  script then filters the data for only the required objects (such as road
-  segments and administrative areas), and extracts the interesting information
-  from those objects.
-* The node geolocations are looked up to generate a geometry for each object.
-  This requires a lot of memory to run efficiently.
-* The geometry is projected to [Web Mercator](https://epsg.io/3857) in this
-  step to avoid continous transformation when tiles are generated later. Most
-  operations will work fine in this projection. Projection is done with the
-  [pyproj](https://pypi.org/project/pyproj/) library.
-* The output is written to a binary file in a very simple format using
-  [msgpack](https://github.com/msgpack/msgpack-python), which is way more
-  efficient that (Geo-)JSON for example. This format is stremable, so the
-  generated file is never fully written or read into memory.
-* The import script reads the msgpack file and sends it to the database using
-  [psycopg](https://www.psycopg.org/). This is done because it supports
-  PostgreSQL's `COPY FROM` statement, which enables much faster writes to the
-  database that a traditionional `INSERT VALUES`. The file is streamed directly
-  to the database, so it is never read into memory.

frontend/src/App.module.less

@@ -69,6 +69,7 @@
 }
 
 .pageTitle a {
+  font-family: 'Open Sans Condensed';
   font-weight: 600;
   font-size: 18pt;
 
@@ -119,15 +120,6 @@
   }
 }
 
-@media @mobile {
-  .menu.menu {
-    > :global(.ui.container) {
-      height: @menuHeightMobile;
-      align-items: stretch;
-    }
-  }
-}
-
 .banner {
   padding: 8px;
   z-index: 100;

frontend/src/App.tsx

@@ -6,7 +6,7 @@ import {BrowserRouter as Router, Switch, Route, Link} from 'react-router-dom'
 import {useObservable} from 'rxjs-hooks'
 import {from} from 'rxjs'
 import {pluck} from 'rxjs/operators'
-import {Helmet} from 'react-helmet'
+import {Helmet} from "react-helmet";
 import {useTranslation} from 'react-i18next'
 
 import {useConfig} from 'config'
@@ -14,7 +14,6 @@ import styles from './App.module.less'
 import {AVAILABLE_LOCALES, setLocale} from 'i18n'
 
 import {
-  AcknowledgementsPage,
   ExportPage,
   HomePage,
   LoginRedirectPage,
@@ -26,7 +25,6 @@ import {
   TrackPage,
   TracksPage,
   UploadPage,
-  MyTracksPage,
 } from 'pages'
 import {Avatar, LoginButton} from 'components'
 import api from 'api'
@@ -79,7 +77,7 @@ const App = connect((state) => ({login: state.login}))(function App({login}) {
         <title>OpenBikeSensor Portal</title>
       </Helmet>
       {config?.banner && <Banner {...config.banner} />}
-      <Menu className={styles.menu} stackable>
+      <Menu className={styles.menu}>
         <Container>
           <Link to="/" component={MenuItemForLink} header className={styles.pageTitle}>
             OpenBikeSensor
@@ -107,12 +105,7 @@ const App = connect((state) => ({login: state.login}))(function App({login}) {
                 </Link>
                 <Dropdown item trigger={<Avatar user={login} className={styles.avatar} />}>
                   <Dropdown.Menu>
-                    <Link
+                    <Link to="/upload" component={DropdownItemForLink} icon="cloud upload" text={t('App.menu.uploadTracks')} />
-                      to="/upload"
-                      component={DropdownItemForLink}
-                      icon="cloud upload"
-                      text={t('App.menu.uploadTracks')}
-                    />
                     <Link to="/settings" component={DropdownItemForLink} icon="cog" text={t('App.menu.settings')}/>
                     <Dropdown.Divider />
                     <Link to="/logout" component={DropdownItemForLink} icon="sign-out" text={t('App.menu.logout')} />
@@ -132,16 +125,14 @@ const App = connect((state) => ({login: state.login}))(function App({login}) {
         <Route path="/" exact>
           <HomePage />
         </Route>
-        {hasMap && (
+        {hasMap && <Route path="/map" exact>
-          <Route path="/map" exact>
           <MapPage />
-          </Route>
+        </Route>}
-        )}
         <Route path="/tracks" exact>
           <TracksPage />
         </Route>
         <Route path="/my/tracks" exact>
-          <MyTracksPage />
+          <TracksPage privateTracks />
         </Route>
         <Route path={`/tracks/:slug`} exact>
           <TrackPage />
@@ -152,9 +143,6 @@ const App = connect((state) => ({login: state.login}))(function App({login}) {
         <Route path="/export" exact>
           <ExportPage />
         </Route>
-        <Route path="/acknowledgements" exact>
-          <AcknowledgementsPage />
-        </Route>
         <Route path="/redirect" exact>
           <LoginRedirectPage />
         </Route>
@@ -181,7 +169,9 @@ const App = connect((state) => ({login: state.login}))(function App({login}) {
           <Grid columns={4} stackable>
             <Grid.Row>
               <Grid.Column>
-                <Header as="h5">{t('App.footer.aboutTheProject')}</Header>
+                <Header as="h5">
+                  {t('App.footer.aboutTheProject')}
+                </Header>
                 <List>
                   <List.Item>
                     <a href="https://openbikesensor.org/" target="_blank" rel="noreferrer">
@@ -192,7 +182,9 @@ const App = connect((state) => ({login: state.login}))(function App({login}) {
               </Grid.Column>
 
               <Grid.Column>
-                <Header as="h5">{t('App.footer.getInvolved')}</Header>
+                <Header as="h5">
+                  {t('App.footer.getInvolved')}
+                </Header>
                 <List>
                   <List.Item>
                     <a href="https://forum.openbikesensor.org/" target="_blank" rel="noreferrer">
@@ -213,7 +205,9 @@ const App = connect((state) => ({login: state.login}))(function App({login}) {
               </Grid.Column>
 
               <Grid.Column>
-                <Header as="h5">{t('App.footer.thisInstallation')}</Header>
+                <Header as="h5">
+                  {t('App.footer.thisInstallation')}
+                </Header>
                 <List>
                   <List.Item>
                     <a href={config?.privacyPolicyUrl} target="_blank" rel="noreferrer">
@@ -225,13 +219,6 @@ const App = connect((state) => ({login: state.login}))(function App({login}) {
                       {t('App.footer.imprint')}
                     </a>
                   </List.Item>
-                  {config?.termsUrl && (
-                    <List.Item>
-                      <a href={config?.termsUrl} target="_blank" rel="noreferrer">
-                        {t('App.footer.terms')}
-                      </a>
-                    </List.Item>
-                  )}
                   <List.Item>
                     <a
                       href={`https://github.com/openbikesensor/portal${
@@ -240,7 +227,7 @@ const App = connect((state) => ({login: state.login}))(function App({login}) {
                       target="_blank"
                       rel="noreferrer"
                     >
-                      {apiVersion ? t('App.footer.version', {apiVersion}) : t('App.footer.versionLoading')}
+                      Version {apiVersion ? t('App.footer.version', {apiVersion}) : t('App.footer.versionLoading')}
                     </a>
                   </List.Item>
                 </List>
@@ -249,11 +236,7 @@ const App = connect((state) => ({login: state.login}))(function App({login}) {
               <Grid.Column>
                 <Header as="h5">{t('App.footer.changeLanguage')}</Header>
                 <List>
-                  {AVAILABLE_LOCALES.map((locale) => (
+                    {AVAILABLE_LOCALES.map(locale => <List.Item key={locale}><a onClick={() => setLocale(locale)}>{t(`locales.${locale}`)}</a></List.Item>)}
-                    <List.Item key={locale}>
-                      <a onClick={() => setLocale(locale)}>{t(`locales.${locale}`)}</a>
-                    </List.Item>
-                  ))}
                 </List>
               </Grid.Column>
             </Grid.Row>

frontend/src/components/Avatar/index.tsx

@@ -1,39 +1,42 @@
-import React from 'react'
+import React from "react";
-import {Comment} from 'semantic-ui-react'
+import { Comment } from "semantic-ui-react";
-import classnames from 'classnames'
+import classnames from "classnames";
 
-import './styles.less'
+import "./styles.less";
 
 function hashCode(s) {
-  let hash = 0
+  let hash = 0;
   for (let i = 0; i < s.length; i++) {
-    hash = (hash << 5) - hash + s.charCodeAt(i)
+    hash = (hash << 5) - hash + s.charCodeAt(i);
-    hash |= 0
+    hash |= 0;
   }
-  return hash
+  return hash;
 }
 
 function getColor(s) {
-  const h = Math.floor(hashCode(s)) % 360
+  const h = Math.floor(hashCode(s)) % 360;
-  return `hsl(${h}, 50%, 50%)`
+  return `hsl(${h}, 50%, 50%)`;
 }
 
 export default function Avatar({ user, className }) {
-  const {image, displayName} = user || {}
+  const { image, displayName } = user || {};
 
   if (image) {
-    return <Comment.Avatar src={image} className={className} />
+    return <Comment.Avatar src={image} className={className} />;
   }
 
   if (!displayName) {
-    return <div className={classnames(className, 'avatar', 'empty-avatar')} />
+    return <div className={classnames(className, "avatar", "empty-avatar")} />;
   }
 
-  const color = getColor(displayName)
+  const color = getColor(displayName);
 
   return (
-    <div className={classnames(className, 'avatar', 'text-avatar')} style={{background: color}}>
+    <div
+      className={classnames(className, "avatar", "text-avatar")}
+      style={{ background: color }}
+    >
       {displayName && <span>{displayName[0]}</span>}
     </div>
-  )
+  );
 }

frontend/src/components/Chart.tsx

@@ -1,7 +1,7 @@
-import React from 'react'
+import React from 'react';
-import ReactEChartsCore from 'echarts-for-react/lib/core'
+import ReactEChartsCore from 'echarts-for-react/lib/core';
 
-import * as echarts from 'echarts/core'
+import * as echarts from 'echarts/core';
 
 import {
   // LineChart,
@@ -26,7 +26,7 @@ import {
   // ThemeRiverChart,
   // SunburstChart,
   // CustomChart,
-} from 'echarts/charts'
+} from 'echarts/charts';
 
 // import components, all suffixed with Component
 import {
@@ -60,18 +60,25 @@ import {
   // AriaComponent,
   // TransformComponent,
   DatasetComponent,
-} from 'echarts/components'
+} from 'echarts/components';
 
 // Import renderer, note that introducing the CanvasRenderer or SVGRenderer is a required step
 import {
   CanvasRenderer,
   // SVGRenderer,
-} from 'echarts/renderers'
+} from 'echarts/renderers';
 
 // Register the required components
-echarts.use([TitleComponent, TooltipComponent, GridComponent, BarChart, CanvasRenderer])
+echarts.use(
+  [TitleComponent, TooltipComponent, GridComponent, BarChart, CanvasRenderer]
+);
 
 // The usage of ReactEChartsCore are same with above.
 export default function Chart(props) {
-  return <ReactEChartsCore echarts={echarts} notMerge lazyUpdate {...props} />
+  return <ReactEChartsCore
+    echarts={echarts}
+    notMerge
+    lazyUpdate
+    {...props}
+  />
 }

frontend/src/components/ColorMapLegend.tsx

@@ -1,18 +1,18 @@
-import React, {useMemo} from 'react'
+import React, {useMemo} from "react";
-
-import styles from './ColorMapLegend.module.less'
 
 type ColorMap = [number, string][]
 
+import styles from './ColorMapLegend.module.less'
+
 function* pairs(arr) {
   for (let i = 1; i < arr.length; i++) {
-    yield [arr[i - 1], arr[i]]
+    yield [arr[i - 1], arr[i]];
   }
 }
 function* zip(...arrs) {
-  const l = Math.min(...arrs.map((a) => a.length))
+  const l = Math.min(...arrs.map(a => a.length));
   for (let i = 0; i < l; i++) {
-    yield arrs.map((a) => a[i])
+    yield arrs.map(a => a[i]);
   }
 }
 
@@ -25,10 +25,10 @@ export function DiscreteColorMapLegend({map}: {map: ColorMap}) {
   min -= buffer
   max += buffer
   const normalizeValue = (v) => (v - min) / (max - min)
-  const stopPairs = Array.from(pairs([min, ...stops, max]))
+  const stopPairs = Array.from(pairs([min, ...stops, max]));
 
-  const gradientId = useMemo(() => `gradient${Math.floor(Math.random() * 1000000)}`, [])
+  const gradientId = useMemo(() => `gradient${Math.floor(Math.random() * 1000000)}`, []);
-  const gradientUrl = `url(#${gradientId})`
+  const gradientUrl = `url(#${gradientId})`;
 
   const parts = Array.from(zip(stopPairs, colors))
 
@@ -42,6 +42,7 @@ export function DiscreteColorMapLegend({map}: {map: ColorMap}) {
               <stop offset={normalizeValue(left) * 100 + '%'} stopColor={color} />
               <stop offset={normalizeValue(right) * 100 + '%'} stopColor={color} />
             </React.Fragment>
+
             ))}
           </linearGradient>
         </defs>
@@ -58,20 +59,12 @@ export function DiscreteColorMapLegend({map}: {map: ColorMap}) {
   )
 }
 
-export default function ColorMapLegend({
+export default function ColorMapLegend({map, twoTicks = false, digits=2}: {map: ColorMap, twoTicks?: boolean, digits?: number}) {
-  map,
-  twoTicks = false,
-  digits = 2,
-}: {
-  map: ColorMap
-  twoTicks?: boolean
-  digits?: number
-}) {
   const min = map[0][0]
   const max = map[map.length - 1][0]
   const normalizeValue = (v) => (v - min) / (max - min)
-  const gradientId = useMemo(() => `gradient${Math.floor(Math.random() * 1000000)}`, [])
+  const gradientId = useMemo(() => `gradient${Math.floor(Math.random() * 1000000)}`, []);
-  const gradientUrl = `url(#${gradientId})`
+  const gradientUrl = `url(#${gradientId})`;
   const tickValues = twoTicks ? [map[0], map[map.length-1]] : map
   return (
     <div className={styles.colorMapLegend}>

frontend/src/components/FileUploadField.tsx

@@ -1,31 +1,31 @@
-import React from 'react'
+import React from "react";
-import {Icon, Segment, Header, Button} from 'semantic-ui-react'
+import { Icon, Segment, Header, Button } from "semantic-ui-react";
-import {useTranslation} from 'react-i18next'
+import { useTranslation } from "react-i18next";
 
-import {FileDrop} from 'components'
+import { FileDrop } from "components";
 
 export default function FileUploadField({ onSelect: onSelect_, multiple }) {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
-  const labelRef = React.useRef()
+  const labelRef = React.useRef();
-  const [labelRefState, setLabelRefState] = React.useState()
+  const [labelRefState, setLabelRefState] = React.useState();
 
-  const onSelect = multiple ? onSelect_ : (files) => onSelect_(files?.[0])
+  const onSelect = multiple ? onSelect_ : (files) => onSelect_(files?.[0]);
 
   React.useLayoutEffect(
     () => {
-      setLabelRefState(labelRef.current)
+      setLabelRefState(labelRef.current);
     },
     // eslint-disable-next-line react-hooks/exhaustive-deps
     [labelRef.current]
-  )
+  );
 
   function onChangeField(e) {
-    e.preventDefault?.()
+    e.preventDefault?.();
 
     if (e.target.files && e.target.files.length) {
-      onSelect(e.target.files)
+      onSelect(e.target.files);
     }
-    e.target.value = '' // reset the form field for uploading again
+    e.target.value = ""; // reset the form field for uploading again
   }
 
   return (
@@ -36,7 +36,7 @@ export default function FileUploadField({onSelect: onSelect_, multiple}) {
         style={{
           width: 0,
           height: 0,
-          position: 'fixed',
+          position: "fixed",
           left: -1000,
           top: -1000,
           opacity: 0.001,
@@ -48,22 +48,34 @@ export default function FileUploadField({onSelect: onSelect_, multiple}) {
       <label htmlFor="upload-field" ref={labelRef}>
         {labelRefState && (
           <FileDrop onDrop={onSelect} frame={labelRefState}>
-            {({draggingOverFrame, draggingOverTarget, onDragOver, onDragLeave, onDrop, onClick}) => (
+            {({
+              draggingOverFrame,
+              draggingOverTarget,
+              onDragOver,
+              onDragLeave,
+              onDrop,
+              onClick,
+            }) => (
               <Segment
                 placeholder
                 {...{ onDragOver, onDragLeave, onDrop }}
                 style={{
-                  background: draggingOverTarget || draggingOverFrame ? '#E0E0EE' : null,
+                  background:
-                  transition: 'background 0.2s',
+                    draggingOverTarget || draggingOverFrame ? "#E0E0EE" : null,
+                  transition: "background 0.2s",
                 }}
               >
                 <Header icon>
                   <Icon name="cloud upload" />
-                  {multiple ? t('FileUploadField.dropOrClickMultiple') : t('FileUploadField.dropOrClick')}
+                  {multiple
+                    ? t("FileUploadField.dropOrClickMultiple")
+                    : t("FileUploadField.dropOrClick")}
                 </Header>
 
                 <Button primary as="span">
-                  {multiple ? t('FileUploadField.uploadFiles') : t('FileUploadField.uploadFile')}
+                  {multiple
+                    ? t("FileUploadField.uploadFiles")
+                    : t("FileUploadField.uploadFile")}
                 </Button>
               </Segment>
             )}
@@ -71,5 +83,5 @@ export default function FileUploadField({onSelect: onSelect_, multiple}) {
         )}
       </label>
     </>
-  )
+  );
 }

frontend/src/components/FormattedDate.tsx

@@ -21,9 +21,5 @@ export default function FormattedDate({date, relative = false}) {
   }
 
   const iso = dateTime.toISO()
-  return (
+  return <time dateTime={iso} title={iso}>{str}</time>
-    <time dateTime={iso} title={iso}>
-      {str}
-    </time>
-  )
 }

frontend/src/components/Map/index.tsx

@@ -6,65 +6,65 @@ import ReactMapGl, {WebMercatorViewport, ScaleControl, NavigationControl, Attrib
 import turfBbox from '@turf/bbox'
 import {useHistory, useLocation} from 'react-router-dom'
 
-import {useConfig} from 'config'
+import { useConfig } from "config";
 
-import {useCallbackRef} from '../../utils'
+import { useCallbackRef } from "../../utils";
-import {baseMapStyles} from '../../mapstyles'
+import { baseMapStyles } from "../../mapstyles";
 
-import styles from './styles.module.less'
+import styles from "./styles.module.less";
 
 interface Viewport {
-  longitude: number
+  longitude: number;
-  latitude: number
+  latitude: number;
-  zoom: number
+  zoom: number;
 }
-const EMPTY_VIEWPORT: Viewport = {longitude: 0, latitude: 0, zoom: 0}
+const EMPTY_VIEWPORT: Viewport = { longitude: 0, latitude: 0, zoom: 0 };
 
 export const withBaseMapStyle = connect((state) => ({
-  baseMapStyle: state.mapConfig?.baseMap?.style ?? 'positron',
+  baseMapStyle: state.mapConfig?.baseMap?.style ?? "positron",
-}))
+}));
 
 function parseHash(v: string): Viewport | null {
-  if (!v) return null
+  if (!v) return null;
-  const m = v.match(/^#([0-9\.]+)\/([0-9\.\-]+)\/([0-9\.\-]+)$/)
+  const m = v.match(/^#([0-9\.]+)\/([0-9\.]+)\/([0-9\.]+)$/);
-  if (!m) return null
+  if (!m) return null;
   return {
     zoom: Number.parseFloat(m[1]),
     latitude: Number.parseFloat(m[2]),
     longitude: Number.parseFloat(m[3]),
-  }
+  };
 }
 
 function buildHash(v: Viewport): string {
-  return `${v.zoom.toFixed(2)}/${v.latitude}/${v.longitude}`
+  return `${v.zoom.toFixed(2)}/${v.latitude}/${v.longitude}`;
 }
 
 const setViewportToHash = _.debounce((history, viewport) => {
   history.replace({
     hash: buildHash(viewport),
-  })
+  });
-}, 200)
+}, 200);
 
 function useViewportFromUrl(): [Viewport | null, (v: Viewport) => void] {
-  const history = useHistory()
+  const history = useHistory();
-  const location = useLocation()
+  const location = useLocation();
 
-  const [cachedValue, setCachedValue] = useState(parseHash(location.hash))
+  const [cachedValue, setCachedValue] = useState(parseHash(location.hash));
 
   // when the location hash changes, set the new value to the cache
   useEffect(() => {
-    setCachedValue(parseHash(location.hash))
+    setCachedValue(parseHash(location.hash));
-  }, [location.hash])
+  }, [location.hash]);
 
   const setter = useCallback(
     (v) => {
-      setCachedValue(v)
+      setCachedValue(v);
-      setViewportToHash(history, v)
+      setViewportToHash(history, v);
     },
     [history]
-  )
+  );
 
-  return [cachedValue || EMPTY_VIEWPORT, setter]
+  return [cachedValue || EMPTY_VIEWPORT, setter];
 }
 
 function Map({
@@ -80,47 +80,54 @@ function Map({
   children: React.ReactNode
   boundsFromJson: GeoJSON.Geometry
   baseMapStyle: string
-  hasToolbar?: boolean
+  hasToolbar?: boolean;
-  onViewportChange: (viewport: Viewport) => void
+  onViewportChange: (viewport: Viewport) => void,
 }) {
-  const [viewportState, setViewportState] = useState(EMPTY_VIEWPORT)
+  const [viewportState, setViewportState] = useState(EMPTY_VIEWPORT);
-  const [viewportUrl, setViewportUrl] = useViewportFromUrl()
+  const [viewportUrl, setViewportUrl] = useViewportFromUrl();
 
   const [viewport, setViewport_] = viewportFromUrl ? [viewportUrl, setViewportUrl] : [viewportState, setViewportState]
-  const setViewport = useCallback(
+  const setViewport = useCallback((viewport: Viewport) => {
-    (viewport: Viewport) => {
+    setViewport_(viewport);
-      setViewport_(viewport)
+    onViewportChange?.(viewport);
-      onViewportChange?.(viewport)
+  }, [setViewport_, onViewportChange])
-    },
-    [setViewport_, onViewportChange]
-  )
 
-  const config = useConfig()
+  const config = useConfig();
   useEffect(() => {
-    if (config?.mapHome && viewport?.latitude === 0 && viewport?.longitude === 0 && !boundsFromJson) {
+    if (
-      setViewport(config.mapHome)
+      config?.mapHome &&
+      viewport?.latitude === 0 &&
+      viewport?.longitude === 0 &&
+      !boundsFromJson
+    ) {
+      setViewport(config.mapHome);
     }
-  }, [config, boundsFromJson])
+  }, [config, boundsFromJson]);
 
   const mapSourceHosts = useMemo(
-    () => _.uniq(config?.obsMapSource?.tiles?.map((tileUrl: string) => new URL(tileUrl).host) ?? []),
+    () =>
+      _.uniq(
+        config?.obsMapSource?.tiles?.map(
+          (tileUrl: string) => new URL(tileUrl).host
+        ) ?? []
+      ),
     [config?.obsMapSource]
-  )
+  );
 
   const transformRequest = useCallbackRef((url, resourceType) => {
-    if (resourceType === 'Tile' && mapSourceHosts.includes(new URL(url).host)) {
+    if (resourceType === "Tile" && mapSourceHosts.includes(new URL(url).host)) {
       return {
         url,
-        credentials: 'include',
+        credentials: "include",
+      };
     }
-    }
+  });
-  })
 
   useEffect(() => {
     if (boundsFromJson) {
-      const bbox = turfBbox(boundsFromJson)
+      const bbox = turfBbox(boundsFromJson);
       if (bbox.every((v) => Math.abs(v) !== Infinity)) {
-        const [minX, minY, maxX, maxY] = bbox
+        const [minX, minY, maxX, maxY] = bbox;
         const vp = new WebMercatorViewport({
           width: 1000,
           height: 800,
@@ -133,11 +140,11 @@ function Map({
             padding: 20,
             offset: [0, -100],
           }
-        )
+        );
-        setViewport(_.pick(vp, ['zoom', 'latitude', 'longitude']))
+        setViewport(_.pick(vp, ["zoom", "latitude", "longitude"]));
       }
     }
-  }, [boundsFromJson])
+  }, [boundsFromJson]);
 
   return (
     <ReactMapGl
@@ -152,11 +159,16 @@ function Map({
       attributionControl={false}
     >
       <AttributionControl style={{ top: 0, right: 0 }} />
-      <NavigationControl showCompass={false} style={{left: 16, top: hasToolbar ? 64 : 16}} />
+      <NavigationControl style={{ left: 16, top: hasToolbar ? 64 : 16 }} />
-      <ScaleControl maxWidth={200} unit="metric" style={{left: 16, bottom: 16}} />
+      <ScaleControl
+        maxWidth={200}
+        unit="metric"
+        style={{ left: 16, bottom: 16 }}
+      />
+
       {children}
     </ReactMapGl>
-  )
+  );
 }
 
-export default withBaseMapStyle(Map)
+export default withBaseMapStyle(Map);

frontend/src/components/Page/index.tsx

@@ -1,9 +1,9 @@
-import React from 'react'
+import React from "react";
-import classnames from 'classnames'
+import classnames from "classnames";
-import {Container} from 'semantic-ui-react'
+import { Container } from "semantic-ui-react";
-import {Helmet} from 'react-helmet'
+import { Helmet } from "react-helmet";
 
-import styles from './Page.module.less'
+import styles from "./Page.module.less";
 
 export default function Page({
   small,
@@ -12,11 +12,11 @@ export default function Page({
   stage,
   title,
 }: {
-  small?: boolean
+  small?: boolean;
-  children: ReactNode
+  children: ReactNode;
-  fullScreen?: boolean
+  fullScreen?: boolean;
-  stage?: ReactNode
+  stage?: ReactNode;
-  title?: string
+  title?: string;
 }) {
   return (
     <>
@@ -37,5 +37,5 @@ export default function Page({
         {fullScreen ? children : <Container>{children}</Container>}
       </main>
     </>
-  )
+  );
 }

frontend/src/components/RegionStats/index.tsx

@@ -1,36 +1,49 @@
-import React, {useState, useCallback} from 'react'
+import React, { useState, useCallback } from "react";
-import {pickBy} from 'lodash'
+import { pickBy } from "lodash";
-import {Loader, Statistic, Pagination, Segment, Header, Menu, Table, Icon} from 'semantic-ui-react'
+import {
-import {useObservable} from 'rxjs-hooks'
+  Loader,
-import {of, from, concat, combineLatest} from 'rxjs'
+  Statistic,
-import {map, switchMap, distinctUntilChanged} from 'rxjs/operators'
+  Pagination,
-import {Duration, DateTime} from 'luxon'
+  Segment,
-
+  Header,
-import api from 'api'
+  Menu,
+  Table,
+  Icon,
+} from "semantic-ui-react";
+import { useObservable } from "rxjs-hooks";
+import { of, from, concat, combineLatest } from "rxjs";
+import { map, switchMap, distinctUntilChanged } from "rxjs/operators";
+import { Duration, DateTime } from "luxon";
 import {useTranslation} from 'react-i18next'
 
+import api from "api";
+
 function formatDuration(seconds) {
   return (
     Duration.fromMillis((seconds ?? 0) * 1000)
-      .as('hours')
+      .as("hours")
-      .toFixed(1) + ' h'
+      .toFixed(1) + " h"
-  )
+  );
 }
 
 export default function Stats() {
   const {t} = useTranslation()
-  const [page, setPage] = useState(1)
-  const PER_PAGE = 10
-  const stats = useObservable(
-    () => of(null).pipe(switchMap(() => concat(of(null), from(api.get('/stats/regions'))))),
-    null
-  )
 
-  const pageCount = stats ? Math.ceil(stats.length / PER_PAGE) : 1
+  const [page, setPage] = useState(1);
+  const PER_PAGE = 10;
+  const stats = useObservable(
+    () =>
+      of(null).pipe(
+        switchMap(() => concat(of(null), from(api.get("/stats/regions"))))
+      ),
+    null
+  );
+
+  const pageCount = stats ? Math.ceil(stats.length / PER_PAGE) : 1;
 
   return (
     <>
-      <Header as="h2">{t('RegionStats.title')}</Header>
+      <Header as="h2">{t(`Stats.topRegions`)}</Header>
 
       <div>
         <Loader active={stats == null} />
@@ -38,13 +51,15 @@ export default function Stats() {
         <Table celled>
           <Table.Header>
             <Table.Row>
-              <Table.HeaderCell> {t('RegionStats.regionName')}</Table.HeaderCell>
+              <Table.HeaderCell>{t(`Stats.regionName`)}</Table.HeaderCell>
-              <Table.HeaderCell>{t('RegionStats.eventCount')}</Table.HeaderCell>
+              <Table.HeaderCell>{t(`Stats.eventCount`)}</Table.HeaderCell>
             </Table.Row>
           </Table.Header>
 
           <Table.Body>
-            {stats?.slice((page - 1) * PER_PAGE, page * PER_PAGE)?.map((area) => (
+            {stats
+              ?.slice((page - 1) * PER_PAGE, page * PER_PAGE)
+              ?.map((area) => (
                 <Table.Row key={area.id}>
                   <Table.Cell>{area.name}</Table.Cell>
                   <Table.Cell>{area.overtaking_event_count}</Table.Cell>
@@ -52,8 +67,7 @@ export default function Stats() {
               ))}
           </Table.Body>
 
-          {pageCount > 1 && (
+    {pageCount > 1 && <Table.Footer>
-            <Table.Footer>
             <Table.Row>
               <Table.HeaderCell colSpan="2">
                 <Pagination
@@ -64,10 +78,9 @@ export default function Stats() {
                 />
               </Table.HeaderCell>
             </Table.Row>
-            </Table.Footer>
+          </Table.Footer>}
-          )}
         </Table>
       </div>
     </>
-  )
+  );
 }

frontend/src/components/Stats/index.tsx

@@ -69,14 +69,14 @@ export default function Stats({user = null}: {user?: null | string}) {
 
   return (
     <>
+      <Header as="h2">{user ? t('Stats.titleUser') : t('Stats.title')}</Header>
+
       <div>
         <Segment attached="top">
           <Loader active={stats == null} />
           <Statistic.Group widths={2} size="tiny">
             <Statistic>
-              <Statistic.Value>
+              <Statistic.Value>{stats ? `${Number(stats?.trackLength / 1000).toFixed(1)} km` : placeholder}</Statistic.Value>
-                {stats ? `${Number(stats?.trackLength / 1000).toFixed(1)} km` : placeholder}
-              </Statistic.Value>
               <Statistic.Label>{t('Stats.totalTrackLength')}</Statistic.Label>
             </Statistic>
             <Statistic>
@@ -87,21 +87,16 @@ export default function Stats({user = null}: {user?: null | string}) {
               <Statistic.Value>{stats?.numEvents ?? placeholder}</Statistic.Value>
               <Statistic.Label>{t('Stats.eventsConfirmed')}</Statistic.Label>
             </Statistic>
+            {user ? (
               <Statistic>
                 <Statistic.Value>{stats?.trackCount ?? placeholder}</Statistic.Value>
                 <Statistic.Label>{t('Stats.tracksRecorded')}</Statistic.Label>
               </Statistic>
-            {!user && (
+            ) : (
-              <>
               <Statistic>
                 <Statistic.Value>{stats?.userCount ?? placeholder}</Statistic.Value>
                 <Statistic.Label>{t('Stats.membersJoined')}</Statistic.Label>
               </Statistic>
-                <Statistic>
-                  <Statistic.Value>{stats?.deviceCount ?? placeholder}</Statistic.Value>
-                  <Statistic.Label>{t('Stats.deviceCount')}</Statistic.Label>
-                </Statistic>
-              </>
             )}
           </Statistic.Group>
         </Segment>

frontend/src/components/Visibility.tsx

@@ -1,14 +1,18 @@
-import React from 'react'
+import React from "react";
-import {Icon} from 'semantic-ui-react'
+import { Icon } from "semantic-ui-react";
-import {useTranslation} from 'react-i18next'
+import { useTranslation } from "react-i18next";
 
 export default function Visibility({ public: public_ }: { public: boolean }) {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
-  const icon = public_ ? <Icon color="blue" name="eye" fitted /> : <Icon name="eye slash" fitted />
+  const icon = public_ ? (
-  const text = public_ ? t('general.public') : t('general.private')
+    <Icon color="blue" name="eye" fitted />
+  ) : (
+    <Icon name="eye slash" fitted />
+  );
+  const text = public_ ? t("general.public") : t("general.private");
   return (
     <>
       {icon} {text}
     </>
-  )
+  );
 }

frontend/src/config.ts

@@ -1,46 +1,45 @@
-import React from 'react'
+import React from "react";
 
 export type MapSource = {
-  type: 'vector'
+  type: "vector";
-  tiles: string[]
+  tiles: string[];
-  minzoom: number
+  minzoom: number;
-  maxzoom: number
+  maxzoom: number;
-}
+};
 
 export interface Config {
-  apiUrl: string
+  apiUrl: string;
   mapHome: {
-    latitude: number
+    latitude: number;
-    longitude: number
+    longitude: number;
-    zoom: number
+    zoom: number;
-  }
+  };
-  obsMapSource?: MapSource
+  obsMapSource?: MapSource;
-  imprintUrl?: string
+  imprintUrl?: string;
-  privacyPolicyUrl?: string
+  privacyPolicyUrl?: string;
-  termsUrl?: string
   banner?: {
-    text: string
+    text: string;
-    style?: 'warning' | 'info'
+    style?: "warning" | "info";
-  }
+  };
 }
 
 async function loadConfig(): Promise<Config> {
-  const response = await fetch(__webpack_public_path__ + 'config.json')
+  const response = await fetch(__webpack_public_path__ + "config.json");
-  const config = await response.json()
+  const config = await response.json();
-  return config
+  return config;
 }
 
-let _configPromise: Promise<Config> = loadConfig()
+let _configPromise: Promise<Config> = loadConfig();
-let _configCache: null | Config = null
+let _configCache: null | Config = null;
 
 export function useConfig() {
-  const [config, setConfig] = React.useState<Config>(_configCache)
+  const [config, setConfig] = React.useState<Config>(_configCache);
   React.useEffect(() => {
     if (!_configCache) {
-      _configPromise.then(setConfig)
+      _configPromise.then(setConfig);
     }
-  }, [])
+  }, []);
-  return config
+  return config;
 }
 
-export default _configPromise
+export default _configPromise;

frontend/src/i18n.ts

@@ -1,87 +1,95 @@
-import {useState, useEffect, useMemo} from 'react'
+import { useState, useEffect, useMemo } from "react";
-import i18next, {TOptions} from 'i18next'
+import i18next, { TOptions } from "i18next";
-import {BehaviorSubject, combineLatest} from 'rxjs'
+import { BehaviorSubject, combineLatest } from "rxjs";
-import {map, distinctUntilChanged} from 'rxjs/operators'
+import { map, distinctUntilChanged } from "rxjs/operators";
-import HttpBackend, {BackendOptions, RequestCallback} from 'i18next-http-backend'
+import HttpBackend, {
-import {initReactI18next} from 'react-i18next'
+  BackendOptions,
-import LanguageDetector from 'i18next-browser-languagedetector'
+  RequestCallback,
+} from "i18next-http-backend";
+import { initReactI18next } from "react-i18next";
+import LanguageDetector from "i18next-browser-languagedetector";
 
-export type AvailableLocales = 'en' | 'de' | 'fr'
+export type AvailableLocales = "en" | "de";
 
-async function request(_options: BackendOptions, url: string, _payload: any, callback: RequestCallback) {
+async function request(
+  _options: BackendOptions,
+  url: string,
+  _payload: any,
+  callback: RequestCallback
+) {
   try {
-    const [lng] = url.split('/')
+    const [lng] = url.split("/");
-    const locale = await import(`translations/${lng}.yaml`)
+    const locale = await import(`translations/${lng}.yaml`);
-    callback(null, {status: 200, data: locale})
+    callback(null, { status: 200, data: locale });
   } catch (e) {
-    console.error(`Unable to load locale at ${url}\n`, e)
+    console.error(`Unable to load locale at ${url}\n`, e);
-    callback(null, {status: 404, data: String(e)})
+    callback(null, { status: 404, data: String(e) });
   }
 }
 
-export const AVAILABLE_LOCALES: AvailableLocales[] = ['en', 'de', 'fr']
+export const AVAILABLE_LOCALES: AvailableLocales[] = ["en", "de"];
 
-const i18n = i18next.createInstance()
+const i18n = i18next.createInstance();
 
 const options: TOptions = {
-  fallbackLng: 'en',
+  fallbackLng: "en",
 
-  ns: ['common'],
+  ns: ["common"],
-  defaultNS: 'common',
+  defaultNS: "common",
   whitelist: AVAILABLE_LOCALES,
 
   // loading via webpack
   backend: {
-    loadPath: '{{lng}}/{{ns}}',
+    loadPath: "{{lng}}/{{ns}}",
     parse: (data: any) => data,
     request,
   },
 
-  load: 'languageOnly',
+  load: "languageOnly",
 
   interpolation: {
     escapeValue: false, // not needed for react as it escapes by default
   },
-}
+};
 
 i18n
   .use(HttpBackend)
   .use(initReactI18next)
   .use(LanguageDetector)
-  .init({...options})
+  .init({ ...options });
 
-const locale$ = new BehaviorSubject<AvailableLocales>('en')
+const locale$ = new BehaviorSubject<AvailableLocales>("en");
 
-export const translate = i18n.t.bind(i18n)
+export const translate = i18n.t.bind(i18n);
 
 export const translate$ = (stringAndData$: [string, any]) =>
   combineLatest([stringAndData$, locale$.pipe(distinctUntilChanged())]).pipe(
     map(([stringAndData]) => {
-      if (typeof stringAndData === 'string') {
+      if (typeof stringAndData === "string") {
-        return i18n.t(stringAndData)
+        return i18n.t(stringAndData);
       } else {
-        const [string, data] = stringAndData
+        const [string, data] = stringAndData;
-        return i18n.t(string, {data})
+        return i18n.t(string, { data });
       }
     })
-  )
+  );
 
 export const setLocale = (locale: AvailableLocales) => {
-  i18n.changeLanguage(locale)
+  i18n.changeLanguage(locale);
-  locale$.next(locale)
+  locale$.next(locale);
-}
+};
 
 export function useLocale() {
-  const [, reload] = useState()
+  const [, reload] = useState();
 
   useEffect(() => {
-    i18n.on('languageChanged', reload)
+    i18n.on("languageChanged", reload);
     return () => {
-      i18n.off('languageChanged', reload)
+      i18n.off("languageChanged", reload);
-    }
+    };
-  }, [])
+  }, []);
 
-  return i18n.language
+  return i18n.language;
 }
 
-export default i18n
+export default i18n;

frontend/src/index.css

@@ -0,0 +1,11 @@
+body {
+  margin: 0;
+  font-family: 'Noto Sans', 'Roboto', -apple-system, BlinkMacSystemFont, 'Segoe UI', 'Oxygen', 'Ubuntu', 'Cantarell',
+    'Fira Sans', 'Droid Sans', 'Helvetica Neue', sans-serif;
+  -webkit-font-smoothing: antialiased;
+  -moz-osx-font-smoothing: grayscale;
+}
+
+code {
+  font-family: 'Noto Sans Mono', source-code-pro, Menlo, Monaco, Consolas, 'Courier New', monospace;
+}

frontend/src/index.js

@@ -3,7 +3,7 @@ import {Settings} from 'luxon'
 import ReactDOM from 'react-dom'
 import 'fomantic-ui-less/semantic.less'
 
-import './index.less'
+import './index.css'
 import App from './App'
 
 import 'maplibre-gl/dist/maplibre-gl.css'

frontend/src/index.less

@@ -1,8 +0,0 @@
-@import 'styles.less';
-
-body {
-  margin: 0;
-  font-family: @fontFamilyDefault;
-  -webkit-font-smoothing: antialiased;
-  -moz-osx-font-smoothing: grayscale;
-}

frontend/src/mapstyles/index.js

@@ -23,13 +23,7 @@ function rgbArrayToColor(arr) {
 }
 
 function rgbArrayToHtml(arr) {
-  return (
+  return "#" + arr.map((v) => Math.round(v * 255).toString(16)).map(v => (v.length == 1 ? '0' : '') + v).join('')
-    '#' +
-    arr
-      .map((v) => Math.round(v * 255).toString(16))
-      .map((v) => (v.length == 1 ? '0' : '') + v)
-      .join('')
-  )
 }
 
 export function colormapToScale(colormap, value, min, max) {
@@ -44,13 +38,17 @@ export function colormapToScale(colormap, value, min, max) {
 export const viridis = simplifyColormap(viridisBase.map(rgbArrayToColor), 20)
 export const viridisSimpleHtml = simplifyColormap(viridisBase.map(rgbArrayToHtml), 10)
 export const grayscale = ['#FFFFFF', '#000000']
-export const reds = ['rgba( 255, 0, 0, 0)', 'rgba( 255, 0, 0, 255)']
+export const reds = [
+  'rgba( 255, 0, 0, 0)',
+  'rgba( 255, 0, 0, 255)',
+]
 
 export function colorByCount(attribute = 'event_count', maxCount, colormap = viridis) {
   return colormapToScale(colormap, ['case', isValidAttribute(attribute), ['get', attribute], 0], 0, maxCount)
 }
 
-var steps = {rural: [1.6, 1.8, 2.0, 2.2], urban: [1.1, 1.3, 1.5, 1.7]}
+var steps = {'rural': [1.6,1.8,2.0,2.2],
+             'urban': [1.1,1.3,1.5,1.7]}
 
 export function isValidAttribute(attribute) {
   if (attribute.endsWith('zone')) {
@@ -60,18 +58,20 @@ export function isValidAttribute(attribute) {
 }
 
 export function borderByZone() {
-  return ['match', ['get', 'zone'], 'rural', 'cyan', 'urban', 'blue', 'purple']
+  return ["match", ['get', 'zone'],
+  "rural", "cyan",
+  "urban", "blue",
+  "purple"
+  ]
 }
 
 export function colorByDistance(attribute = 'distance_overtaker_mean', fallback = '#ABC', zone='urban') {
+
   return [
     'case',
     ['!', isValidAttribute(attribute)],
     fallback,
-    [
+    ["match", ['get', 'zone'], "rural",
-      'match',
-      ['get', 'zone'],
-      'rural',
     [
       'step',
       ['get', attribute],
@@ -84,8 +84,7 @@ export function colorByDistance(attribute = 'distance_overtaker_mean', fallback
       'rgba(67, 200, 0, 1)',
       steps['rural'][3],
       'rgba(67, 150, 0, 1)',
-      ],
+    ], "urban",
-      'urban',
     [
       'step',
       ['get', attribute],
@@ -111,8 +110,8 @@ export function colorByDistance(attribute = 'distance_overtaker_mean', fallback
       'rgba(67, 200, 0, 1)',
       steps['urban'][3],
       'rgba(67, 150, 0, 1)',
-      ],
+    ]
-    ],
+    ]
   ]
 }
 
@@ -125,57 +124,61 @@ export const trackLayer = {
   },
 }
 
-export const getRegionLayers = (adminLevel = 6, baseColor = '#00897B', maxValue = 5000) => [
+export const getRegionLayers = (adminLevel = 6, baseColor = "#00897B", maxValue = 5000) => [{
-  {
   id: 'region',
-    type: 'fill',
+  "type": "fill",
-    source: 'obs',
+  "source": "obs",
-    'source-layer': 'obs_regions',
+  "source-layer": "obs_regions",
-    minzoom: 0,
+  "minzoom": 0,
-    maxzoom: 10,
+  "maxzoom": 11,
-    // filter: [">", "overtaking_event_count", 0],
+  "filter": [
-    paint: {
+    "all",
-      'fill-color': baseColor,
+    ["==", "admin_level", adminLevel],
-      'fill-antialias': true,
+    [">", "overtaking_event_count", 0],
-      'fill-opacity': [
+  ],
-        'interpolate',
+  "paint": {
-        ['linear'],
+    "fill-color": baseColor,
-        ['log10', ['max', ['get', 'overtaking_event_count'], 1]],
+    "fill-antialias": true,
+    "fill-opacity": [
+      "interpolate",
+      ["linear"],
+      [
+        "log10",
+        [
+          "get",
+          "overtaking_event_count"
+        ]
+      ],
       0,
       0,
       Math.log10(maxValue),
-        0.9,
+      0.9
-      ],
+    ]
   },
 },
 {
   id: 'region-border',
-    type: 'line',
+  "type": "line",
-    source: 'obs',
+  "source": "obs",
-    'source-layer': 'obs_regions',
+  "source-layer": "obs_regions",
-    minzoom: 0,
+  "minzoom": 0,
-    maxzoom: 10,
+  "maxzoom": 11,
-    // filter: [">", "overtaking_event_count", 0],
+  "filter": [
-    paint: {
+    "all",
-      'line-width': [
+    ["==", "admin_level", adminLevel],
-        'interpolate',
+    [">", "overtaking_event_count", 0],
-        ['linear'],
-        ['log10', ['max', ['get', 'overtaking_event_count'], 1]],
-        0,
-        0.2,
-        Math.log10(maxValue),
-        1.5,
   ],
-      'line-color': baseColor,
+  "paint": {
+    "line-width": 1,
+    "line-color": baseColor,
   },
-    layout: {
+  "layout": {
-      'line-join': 'round',
+    "line-join": "round",
-      'line-cap': 'round',
+    "line-cap": "round"
-    },
+  }
-  },
+}]
-]
 
-export const trackLayerRaw = produce(trackLayer, (draft) => {
+export const trackLayerRaw = produce(trackLayer, draft => {
   // draft.paint['line-color'] = '#81D4FA'
   draft.paint['line-width'][4] = 1
   draft.paint['line-width'][6] = 2

frontend/src/pages/AcknowledgementsPage.tsx

@@ -1,18 +0,0 @@
-import React from 'react'
-import {Header} from 'semantic-ui-react'
-import {useTranslation} from 'react-i18next'
-import Markdown from 'react-markdown'
-
-import {Page} from 'components'
-
-export default function AcknowledgementsPage() {
-  const {t} = useTranslation()
-  const title = t('AcknowledgementsPage.title')
-
-  return (
-    <Page title={title}>
-      <Header as="h2">{title}</Header>
-      <Markdown>{t('AcknowledgementsPage.information')}</Markdown>
-    </Page>
-  )
-}

frontend/src/pages/ExportPage/index.tsx

@@ -1,42 +1,54 @@
-import React, {useState, useCallback, useMemo} from 'react'
+import React, { useState, useCallback, useMemo } from "react";
-import {Source, Layer} from 'react-map-gl'
+import { Source, Layer } from "react-map-gl";
-import _ from 'lodash'
+import _ from "lodash";
-import {Button, Form, Dropdown, Header, Message, Icon} from 'semantic-ui-react'
+import {
-import {useTranslation, Trans as Translate} from 'react-i18next'
+  Button,
-import Markdown from 'react-markdown'
+  Form,
+  Dropdown,
+  Header,
+  Message,
+  Icon,
+} from "semantic-ui-react";
+import { useTranslation, Trans as Translate } from "react-i18next";
+import Markdown from "react-markdown";
 
-import {useConfig} from 'config'
+import { useConfig } from "config";
-import {Page, Map} from 'components'
+import { Page, Map } from "components";
 
-const BoundingBoxSelector = React.forwardRef(({value, name, onChange}, ref) => {
+const BoundingBoxSelector = React.forwardRef(
-  const {t} = useTranslation()
+  ({ value, name, onChange }, ref) => {
-  const [pointNum, setPointNum] = useState(0)
+    const { t } = useTranslation();
-  const [point0, setPoint0] = useState(null)
+    const [pointNum, setPointNum] = useState(0);
-  const [point1, setPoint1] = useState(null)
+    const [point0, setPoint0] = useState(null);
+    const [point1, setPoint1] = useState(null);
 
     const onClick = (e) => {
       if (pointNum == 0) {
-      setPoint0(e.lngLat)
+        setPoint0(e.lngLat);
       } else {
-      setPoint1(e.lngLat)
+        setPoint1(e.lngLat);
-    }
-    setPointNum(1 - pointNum)
       }
+      setPointNum(1 - pointNum);
+    };
 
     React.useEffect(() => {
-    if (!point0 || !point1) return
+      if (!point0 || !point1) return;
-    const bbox = `${point0[0]},${point0[1]},${point1[0]},${point1[1]}`
+      const bbox = `${point0[0]},${point0[1]},${point1[0]},${point1[1]}`;
       if (bbox !== value) {
-      onChange(bbox)
+        onChange(bbox);
       }
-  }, [point0, point1])
+    }, [point0, point1]);
 
     React.useEffect(() => {
-    if (!value) return
+      if (!value) return;
-    const [p00, p01, p10, p11] = value.split(',').map((v) => Number.parseFloat(v))
+      const [p00, p01, p10, p11] = value
-    if (!point0 || point0[0] != p00 || point0[1] != p01) setPoint0([p00, p01])
+        .split(",")
-    if (!point1 || point1[0] != p10 || point1[1] != p11) setPoint1([p10, p11])
+        .map((v) => Number.parseFloat(v));
-  }, [value])
+      if (!point0 || point0[0] != p00 || point0[1] != p01)
+        setPoint0([p00, p01]);
+      if (!point1 || point1[0] != p10 || point1[1] != p11)
+        setPoint1([p10, p11]);
+    }, [value]);
 
     return (
       <div>
@@ -46,7 +58,7 @@ const BoundingBoxSelector = React.forwardRef(({value, name, onChange}, ref) => {
           onChange={(e) => onChange(e.target.value)}
         />
 
-      <div style={{height: 400, position: 'relative', marginBottom: 16}}>
+        <div style={{ height: 400, position: "relative", marginBottom: 16 }}>
           <Map onClick={onClick}>
             <Source
               id="bbox"
@@ -54,12 +66,12 @@ const BoundingBoxSelector = React.forwardRef(({value, name, onChange}, ref) => {
               data={
                 point0 && point1
                   ? {
-                    type: 'FeatureCollection',
+                      type: "FeatureCollection",
                       features: [
                         {
-                        type: 'Feature',
+                          type: "Feature",
                           geometry: {
-                          type: 'Polygon',
+                            type: "Polygon",
                             coordinates: [
                               [
                                 [point0[0], point0[1]],
@@ -80,42 +92,44 @@ const BoundingBoxSelector = React.forwardRef(({value, name, onChange}, ref) => {
                 id="bbox"
                 type="line"
                 paint={{
-                'line-width': 4,
+                  "line-width": 4,
-                'line-color': '#F06292',
+                  "line-color": "#F06292",
                 }}
               />
             </Source>
           </Map>
         </div>
       </div>
-  )
+    );
-})
+  }
+);
 
-const MODES = ['events', 'segments']
+const MODES = ["events"];
-const FORMATS = ['geojson', 'shapefile']
+const FORMATS = ["geojson", "shapefile"];
 
 export default function ExportPage() {
-  const [mode, setMode] = useState('events')
+  const [mode, setMode] = useState("events");
-  const [bbox, setBbox] = useState('8.294678,49.651182,9.059601,50.108249')
+  const [bbox, setBbox] = useState("8.294678,49.651182,9.059601,50.108249");
-  const [fmt, setFmt] = useState('geojson')
+  const [fmt, setFmt] = useState("geojson");
-  const config = useConfig()
+  const config = useConfig();
-  const {t} = useTranslation()
+  const exportUrl = `${config?.apiUrl}/export/events?bbox=${bbox}&fmt=${fmt}`;
+  const { t } = useTranslation();
   return (
     <Page title="Export">
-      <Header as="h2">{t('ExportPage.title')}</Header>
+      <Header as="h2">{t("ExportPage.title")}</Header>
 
       <Message icon info>
         <Icon name="info circle" />
         <Message.Content>
-          <Markdown>{t('ExportPage.information')}</Markdown>
+          <Markdown>{t("ExportPage.information")}</Markdown>
         </Message.Content>
       </Message>
 
       <Form>
         <Form.Field>
-          <label>{t('ExportPage.mode.label')}</label>
+          <label>{t("ExportPage.mode.label")}</label>
           <Dropdown
-            placeholder={t('ExportPage.mode.placeholder')}
+            placeholder={t("ExportPage.mode.placeholder")}
             fluid
             selection
             options={MODES.map((value) => ({
@@ -129,9 +143,9 @@ export default function ExportPage() {
         </Form.Field>
 
         <Form.Field>
-          <label>{t('ExportPage.format.label')}</label>
+          <label>{t("ExportPage.format.label")}</label>
           <Dropdown
-            placeholder={t('ExportPage.format.placeholder')}
+            placeholder={t("ExportPage.format.placeholder")}
             fluid
             selection
             options={FORMATS.map((value) => ({
@@ -149,7 +163,7 @@ export default function ExportPage() {
         <Button
           primary
           as="a"
-          href={`${config?.apiUrl}/export/${mode}?bbox=${bbox}&fmt=${fmt}`}
+          href={exportUrl}
           target="_blank"
           rel="noreferrer noopener"
         >
@@ -157,5 +171,5 @@ export default function ExportPage() {
         </Button>
       </Form>
     </Page>
-  )
+  );
 }

frontend/src/pages/HomePage.tsx

@@ -46,10 +46,10 @@ export default function HomePage() {
         <Grid.Row>
           <Grid.Column width={8}>
             <Stats />
-            <MostRecentTrack />
+            <RegionStats />
           </Grid.Column>
           <Grid.Column width={8}>
-            <RegionStats />
+            <MostRecentTrack />
           </Grid.Column>
         </Grid.Row>
       </Grid>

frontend/src/pages/LoginRedirectPage.tsx

@@ -1,66 +1,69 @@
-import React from 'react'
+import React from "react";
-import {connect} from 'react-redux'
+import { connect } from "react-redux";
-import {Redirect, useLocation, useHistory} from 'react-router-dom'
+import { Redirect, useLocation, useHistory } from "react-router-dom";
-import {Icon, Message} from 'semantic-ui-react'
+import { Icon, Message } from "semantic-ui-react";
-import {useObservable} from 'rxjs-hooks'
+import { useObservable } from "rxjs-hooks";
-import {switchMap, pluck, distinctUntilChanged} from 'rxjs/operators'
+import { switchMap, pluck, distinctUntilChanged } from "rxjs/operators";
-import {useTranslation} from 'react-i18next'
+import { useTranslation } from "react-i18next";
 
-import {Page} from 'components'
+import { Page } from "components";
-import api from 'api'
+import api from "api";
 
 const LoginRedirectPage = connect((state) => ({
   loggedIn: Boolean(state.login),
 }))(function LoginRedirectPage({ loggedIn }) {
-  const location = useLocation()
+  const location = useLocation();
-  const history = useHistory()
+  const history = useHistory();
-  const {search} = location
+  const { search } = location;
-  const {t} = useTranslation()
+  const { t } = useTranslation();
 
   /* eslint-disable react-hooks/exhaustive-deps */
 
   // Hook dependency arrays in this block are intentionally left blank, we want
   // to keep the initial state, but reset the url once, ASAP, to not leak the
   // query parameters. This is considered good practice by OAuth.
-  const searchParams = React.useMemo(() => Object.fromEntries(new URLSearchParams(search).entries()), [])
+  const searchParams = React.useMemo(
+    () => Object.fromEntries(new URLSearchParams(search).entries()),
+    []
+  );
 
   React.useEffect(() => {
-    history.replace({...location, search: ''})
+    history.replace({ ...location, search: "" });
-  }, [])
+  }, []);
   /* eslint-enable react-hooks/exhaustive-deps */
 
   if (loggedIn) {
-    return <Redirect to="/" />
+    return <Redirect to="/" />;
   }
 
-  const {error, error_description: errorDescription, code} = searchParams
+  const { error, error_description: errorDescription, code } = searchParams;
 
   if (error) {
     return (
-      <Page small title={t('LoginRedirectPage.loginError')}>
+      <Page small title={t("LoginRedirectPage.loginError")}>
         <LoginError errorText={errorDescription || error} />
       </Page>
-    )
+    );
   }
 
-  return <ExchangeAuthCode code={code} />
+  return <ExchangeAuthCode code={code} />;
-})
+});
 
 function LoginError({ errorText }: { errorText: string }) {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
   return (
     <Message icon error>
       <Icon name="warning sign" />
       <Message.Content>
-        <Message.Header>{t('LoginRedirectPage.loginError')}</Message.Header>
+        <Message.Header>{t("LoginRedirectPage.loginError")}</Message.Header>
-        {t('LoginRedirectPage.loginErrorText', {error: errorText})}
+        {t("LoginRedirectPage.loginErrorText", { error: errorText })}
       </Message.Content>
     </Message>
-  )
+  );
 }
 
 function ExchangeAuthCode({ code }) {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
   const result = useObservable(
     (_$, args$) =>
       args$.pipe(
@@ -70,31 +73,31 @@ function ExchangeAuthCode({code}) {
       ),
     null,
     [code]
-  )
+  );
 
-  let content
+  let content;
   if (result === null) {
     content = (
       <Message icon info>
         <Icon name="circle notched" loading />
         <Message.Content>
-          <Message.Header>{t('LoginRedirectPage.loggingIn')}</Message.Header>
+          <Message.Header>{t("LoginRedirectPage.loggingIn")}</Message.Header>
-          {t('LoginRedirectPage.hangTight')}
+          {t("LoginRedirectPage.hangTight")}
         </Message.Content>
       </Message>
-    )
+    );
   } else if (result === true) {
-    content = <Redirect to="/" />
+    content = <Redirect to="/" />;
   } else {
-    const {error, error_description: errorDescription} = result
+    const { error, error_description: errorDescription } = result;
-    content = <LoginError errorText={errorDescription || error} />
+    content = <LoginError errorText={errorDescription || error} />;
   }
 
   return (
     <Page small title="Login">
       {content}
     </Page>
-  )
+  );
 }
 
-export default LoginRedirectPage
+export default LoginRedirectPage;

frontend/src/pages/MapPage/LayerSidebar.tsx

@@ -1,65 +1,71 @@
-import React from 'react'
+import React from "react";
-import _ from 'lodash'
+import _ from "lodash";
-import {connect} from 'react-redux'
+import { connect } from "react-redux";
-import {Link} from 'react-router-dom'
+import {
-import {List, Select, Input, Divider, Label, Checkbox, Header} from 'semantic-ui-react'
+  List,
-import {useTranslation} from 'react-i18next'
+  Select,
+  Input,
+  Divider,
+  Label,
+  Checkbox,
+  Header,
+} from "semantic-ui-react";
+import { useTranslation } from "react-i18next";
+
 
 import {
   MapConfig,
   setMapConfigFlag as setMapConfigFlagAction,
   initialState as defaultMapConfig,
-} from 'reducers/mapConfig'
+} from "reducers/mapConfig";
-import {colorByDistance, colorByCount, viridisSimpleHtml} from 'mapstyles'
+import { colorByDistance, colorByCount, viridisSimpleHtml } from "mapstyles";
-import {ColorMapLegend, DiscreteColorMapLegend} from 'components'
+import { ColorMapLegend, DiscreteColorMapLegend } from "components";
-import styles from './styles.module.less'
 
-const BASEMAP_STYLE_OPTIONS = ['positron', 'bright']
+const BASEMAP_STYLE_OPTIONS = ["positron", "bright"];
 
 const ROAD_ATTRIBUTE_OPTIONS = [
-  'distance_overtaker_mean',
+  "distance_overtaker_mean",
-  'distance_overtaker_min',
+  "distance_overtaker_min",
-  'distance_overtaker_max',
+  "distance_overtaker_max",
-  'distance_overtaker_median',
+  "distance_overtaker_median",
-  'overtaking_event_count',
+  "overtaking_event_count",
-  'usage_count',
+  "usage_count",
-  'zone',
+  "zone",
-]
+];
 
-const DATE_FILTER_MODES = ['none', 'range', 'threshold']
+const DATE_FILTER_MODES = ["none", "range", "threshold"];
 
-type User = Object
+type User = Object;
 
 function LayerSidebar({
   mapConfig,
   login,
   setMapConfigFlag,
 }: {
-  login: User | null
+  login: User | null;
-  mapConfig: MapConfig
+  mapConfig: MapConfig;
-  setMapConfigFlag: (flag: string, value: unknown) => void
+  setMapConfigFlag: (flag: string, value: unknown) => void;
 }) {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
   const {
     baseMap: { style },
     obsRoads: { show: showRoads, showUntagged, attribute, maxCount },
     obsEvents: { show: showEvents },
     obsRegions: {show: showRegions},
-    filters: {currentUser: filtersCurrentUser, dateMode, startDate, endDate, thresholdAfter},
+    filters: {
+      currentUser: filtersCurrentUser,
+      dateMode,
+      startDate,
+      endDate,
+      thresholdAfter,
+    },
   } = mapConfig
 
-  const openStreetMapCopyright = (
-    <List.Item className={styles.copyright}>
-      {t('MapPage.sidebar.copyright.openStreetMap')}{' '}
-      <Link to="/acknowledgements">{t('MapPage.sidebar.copyright.learnMore')}</Link>
-    </List.Item>
-  )
-
   return (
     <div>
       <List relaxed>
         <List.Item>
-          <List.Header>{t('MapPage.sidebar.baseMap.style.label')}</List.Header>
+          <List.Header>{t("MapPage.sidebar.baseMap.style.label")}</List.Header>
           <Select
             options={BASEMAP_STYLE_OPTIONS.map((value) => ({
               value,
@@ -67,10 +73,11 @@ function LayerSidebar({
               text: t(`MapPage.sidebar.baseMap.style.${value}`),
             }))}
             value={style}
-            onChange={(_e, {value}) => setMapConfigFlag('baseMap.style', value)}
+            onChange={(_e, { value }) =>
+              setMapConfigFlag("baseMap.style", value)
+            }
           />
         </List.Item>
-        {openStreetMapCopyright}
         <Divider />
         <List.Item>
           <Checkbox
@@ -82,25 +89,14 @@ function LayerSidebar({
             onChange={() => setMapConfigFlag('obsRegions.show', !showRegions)}
           />
           <label htmlFor="obsRegions.show">
-            <Header as="h4">{t('MapPage.sidebar.obsRegions.title')}</Header>
+            <Header as="h4">Regions</Header>
           </label>
         </List.Item>
         {showRegions && (
           <>
-            <List.Item>{t('MapPage.sidebar.obsRegions.colorByEventCount')}</List.Item>
+            <List.Item>Color regions based on event count</List.Item>
             <List.Item>
-              <ColorMapLegend
+              <ColorMapLegend twoTicks map={[[0, "#00897B00"], [5000, "#00897BFF"]]} digits={0} />
-                twoTicks
-                map={[
-                  [0, '#00897B00'],
-                  [5000, '#00897BFF'],
-                ]}
-                digits={0}
-              />
-            </List.Item>
-            <List.Item className={styles.copyright}>
-              {t('MapPage.sidebar.copyright.boundaries')}{' '}
-              <Link to="/acknowledgements">{t('MapPage.sidebar.copyright.learnMore')}</Link>
             </List.Item>
           </>
         )}
@@ -110,12 +106,12 @@ function LayerSidebar({
             toggle
             size="small"
             id="obsRoads.show"
-            style={{float: 'right'}}
+            style={{ float: "right" }}
             checked={showRoads}
-            onChange={() => setMapConfigFlag('obsRoads.show', !showRoads)}
+            onChange={() => setMapConfigFlag("obsRoads.show", !showRoads)}
           />
           <label htmlFor="obsRoads.show">
-            <Header as="h4">{t('MapPage.sidebar.obsRoads.title')}</Header>
+            <Header as="h4">{t("MapPage.sidebar.obsRoads.title")}</Header>
           </label>
         </List.Item>
         {showRoads && (
@@ -123,12 +119,16 @@ function LayerSidebar({
             <List.Item>
               <Checkbox
                 checked={showUntagged}
-                onChange={() => setMapConfigFlag('obsRoads.showUntagged', !showUntagged)}
+                onChange={() =>
-                label={t('MapPage.sidebar.obsRoads.showUntagged.label')}
+                  setMapConfigFlag("obsRoads.showUntagged", !showUntagged)
+                }
+                label={t("MapPage.sidebar.obsRoads.showUntagged.label")}
               />
             </List.Item>
             <List.Item>
-              <List.Header>{t('MapPage.sidebar.obsRoads.attribute.label')}</List.Header>
+              <List.Header>
+                {t("MapPage.sidebar.obsRoads.attribute.label")}
+              </List.Header>
               <Select
                 fluid
                 options={ROAD_ATTRIBUTE_OPTIONS.map((value) => ({
@@ -137,54 +137,77 @@ function LayerSidebar({
                   text: t(`MapPage.sidebar.obsRoads.attribute.${value}`),
                 }))}
                 value={attribute}
-                onChange={(_e, {value}) => setMapConfigFlag('obsRoads.attribute', value)}
+                onChange={(_e, { value }) =>
+                  setMapConfigFlag("obsRoads.attribute", value)
+                }
               />
             </List.Item>
-            {attribute.endsWith('_count') ? (
+            {attribute.endsWith("_count") ? (
               <>
                 <List.Item>
-                  <List.Header>{t('MapPage.sidebar.obsRoads.maxCount.label')}</List.Header>
+                  <List.Header>
+                    {t("MapPage.sidebar.obsRoads.maxCount.label")}
+                  </List.Header>
                   <Input
                     fluid
                     type="number"
                     value={maxCount}
-                    onChange={(_e, {value}) => setMapConfigFlag('obsRoads.maxCount', value)}
+                    onChange={(_e, { value }) =>
+                      setMapConfigFlag("obsRoads.maxCount", value)
+                    }
                   />
                 </List.Item>
                 <List.Item>
                   <ColorMapLegend
                     map={_.chunk(
-                      colorByCount('obsRoads.maxCount', mapConfig.obsRoads.maxCount, viridisSimpleHtml).slice(3),
+                      colorByCount(
+                        "obsRoads.maxCount",
+                        mapConfig.obsRoads.maxCount,
+                        viridisSimpleHtml
+                      ).slice(3),
                       2
                     )}
                     twoTicks
                   />
                 </List.Item>
               </>
-            ) : attribute.endsWith('zone') ? (
+            ) : attribute.endsWith("zone") ? (
               <>
                 <List.Item>
-                  <Label size="small" style={{background: 'blue', color: 'white'}}>
+                  <Label
-                    {t('general.zone.urban')} (1.5&nbsp;m)
+                    size="small"
+                    style={{ background: "blue", color: "white" }}
+                  >
+                    {t("general.zone.urban")} (1.5&nbsp;m)
                   </Label>
-                  <Label size="small" style={{background: 'cyan', color: 'black'}}>
+                  <Label
-                    {t('general.zone.rural')}(2&nbsp;m)
+                    size="small"
+                    style={{ background: "cyan", color: "black" }}
+                  >
+                    {t("general.zone.rural")}(2&nbsp;m)
                   </Label>
                 </List.Item>
               </>
             ) : (
               <>
                 <List.Item>
-                  <List.Header>{_.upperFirst(t('general.zone.urban'))}</List.Header>
+                  <List.Header>
-                  <DiscreteColorMapLegend map={colorByDistance('distance_overtaker')[3][5].slice(2)} />
+                    {_.upperFirst(t("general.zone.urban"))}
+                  </List.Header>
+                  <DiscreteColorMapLegend
+                    map={colorByDistance("distance_overtaker")[3][5].slice(2)}
+                  />
                 </List.Item>
                 <List.Item>
-                  <List.Header>{_.upperFirst(t('general.zone.rural'))}</List.Header>
+                  <List.Header>
-                  <DiscreteColorMapLegend map={colorByDistance('distance_overtaker')[3][3].slice(2)} />
+                    {_.upperFirst(t("general.zone.rural"))}
+                  </List.Header>
+                  <DiscreteColorMapLegend
+                    map={colorByDistance("distance_overtaker")[3][3].slice(2)}
+                  />
                 </List.Item>
               </>
             )}
-            {openStreetMapCopyright}
           </>
         )}
         <Divider />
@@ -193,36 +216,40 @@ function LayerSidebar({
             toggle
             size="small"
             id="obsEvents.show"
-            style={{float: 'right'}}
+            style={{ float: "right" }}
             checked={showEvents}
-            onChange={() => setMapConfigFlag('obsEvents.show', !showEvents)}
+            onChange={() => setMapConfigFlag("obsEvents.show", !showEvents)}
           />
           <label htmlFor="obsEvents.show">
-            <Header as="h4">{t('MapPage.sidebar.obsEvents.title')}</Header>
+            <Header as="h4">{t("MapPage.sidebar.obsEvents.title")}</Header>
           </label>
         </List.Item>
         {showEvents && (
           <>
             <List.Item>
-              <List.Header>{_.upperFirst(t('general.zone.urban'))}</List.Header>
+              <List.Header>{_.upperFirst(t("general.zone.urban"))}</List.Header>
-              <DiscreteColorMapLegend map={colorByDistance('distance_overtaker')[3][5].slice(2)} />
+              <DiscreteColorMapLegend
+                map={colorByDistance("distance_overtaker")[3][5].slice(2)}
+              />
             </List.Item>
             <List.Item>
-              <List.Header>{_.upperFirst(t('general.zone.rural'))}</List.Header>
+              <List.Header>{_.upperFirst(t("general.zone.rural"))}</List.Header>
-              <DiscreteColorMapLegend map={colorByDistance('distance_overtaker')[3][3].slice(2)} />
+              <DiscreteColorMapLegend
+                map={colorByDistance("distance_overtaker")[3][3].slice(2)}
+              />
             </List.Item>
           </>
         )}
         <Divider />
 
         <List.Item>
-          <Header as="h4">{t('MapPage.sidebar.filters.title')}</Header>
+          <Header as="h4">{t("MapPage.sidebar.filters.title")}</Header>
         </List.Item>
 
         {login && (
           <>
             <List.Item>
-              <Header as="h5">{t('MapPage.sidebar.filters.userData')}</Header>
+              <Header as="h5">{t("MapPage.sidebar.filters.userData")}</Header>
             </List.Item>
 
             <List.Item>
@@ -231,13 +258,15 @@ function LayerSidebar({
                 size="small"
                 id="filters.currentUser"
                 checked={filtersCurrentUser}
-                onChange={() => setMapConfigFlag('filters.currentUser', !filtersCurrentUser)}
+                onChange={() =>
-                label={t('MapPage.sidebar.filters.currentUser')}
+                  setMapConfigFlag("filters.currentUser", !filtersCurrentUser)
+                }
+                label={t("MapPage.sidebar.filters.currentUser")}
               />
             </List.Item>
 
             <List.Item>
-              <Header as="h5">{t('MapPage.sidebar.filters.dateRange')}</Header>
+              <Header as="h5">{t("MapPage.sidebar.filters.dateRange")}</Header>
             </List.Item>
 
             <List.Item>
@@ -248,12 +277,14 @@ function LayerSidebar({
                   key: value,
                   text: t(`MapPage.sidebar.filters.dateMode.${value}`),
                 }))}
-                value={dateMode ?? 'none'}
+                value={dateMode ?? "none"}
-                onChange={(_e, {value}) => setMapConfigFlag('filters.dateMode', value)}
+                onChange={(_e, { value }) =>
+                  setMapConfigFlag("filters.dateMode", value)
+                }
               />
             </List.Item>
 
-            {dateMode == 'range' && (
+            {dateMode == "range" && (
               <List.Item>
                 <Input
                   type="date"
@@ -261,14 +292,16 @@ function LayerSidebar({
                   step="7"
                   size="small"
                   id="filters.startDate"
-                  onChange={(_e, {value}) => setMapConfigFlag('filters.startDate', value)}
+                  onChange={(_e, { value }) =>
+                    setMapConfigFlag("filters.startDate", value)
+                  }
                   value={startDate ?? null}
-                  label={t('MapPage.sidebar.filters.start')}
+                  label={t("MapPage.sidebar.filters.start")}
                 />
             </List.Item>
             )}
 
-            {dateMode == 'range' && (
+            {dateMode == "range" && (
               <List.Item>
                 <Input
                   type="date"
@@ -276,14 +309,16 @@ function LayerSidebar({
                   step="7"
                   size="small"
                   id="filters.endDate"
-                  onChange={(_e, {value}) => setMapConfigFlag('filters.endDate', value)}
+                  onChange={(_e, { value }) =>
+                    setMapConfigFlag("filters.endDate", value)
+                  }
                   value={endDate ?? null}
-                  label={t('MapPage.sidebar.filters.end')}
+                  label={t("MapPage.sidebar.filters.end")}
                 />
               </List.Item>
             )}
 
-            {dateMode == 'threshold' && (
+            {dateMode == "threshold" && (
               <List.Item>
                 <Input
                   type="date"
@@ -292,33 +327,42 @@ function LayerSidebar({
                   size="small"
                   id="filters.startDate"
                   value={startDate ?? null}
-                  onChange={(_e, {value}) => setMapConfigFlag('filters.startDate', value)}
+                  onChange={(_e, { value }) =>
-                  label={t('MapPage.sidebar.filters.threshold')}
+                    setMapConfigFlag("filters.startDate", value)
+                  }
+                  label={t("MapPage.sidebar.filters.threshold")}
                 />
               </List.Item>
             )}
 
-            {dateMode == 'threshold' && (
+            {dateMode == "threshold" && (
               <List.Item>
                 <span>
-                  {t('MapPage.sidebar.filters.before')}{' '}
+                  {t("MapPage.sidebar.filters.before")}{" "}
                   <Checkbox
                     toggle
                     size="small"
                     checked={thresholdAfter ?? false}
-                    onChange={() => setMapConfigFlag('filters.thresholdAfter', !thresholdAfter)}
+                    onChange={() =>
+                      setMapConfigFlag(
+                        "filters.thresholdAfter",
+                        !thresholdAfter
+                      )
+                    }
                     id="filters.thresholdAfter"
-                  />{' '}
+                  />{" "}
-                  {t('MapPage.sidebar.filters.after')}
+                  {t("MapPage.sidebar.filters.after")}
                 </span>
               </List.Item>
             )}
           </>
         )}
-        {!login && <List.Item>{t('MapPage.sidebar.filters.needsLogin')}</List.Item>}
+        {!login && (
+          <List.Item>{t("MapPage.sidebar.filters.needsLogin")}</List.Item>
+        )}
       </List>
     </div>
-  )
+  );
 }
 
 export default connect(
@@ -333,4 +377,4 @@ export default connect(
   }),
   { setMapConfigFlag: setMapConfigFlagAction }
   //
-)(LayerSidebar)
+)(LayerSidebar);

frontend/src/pages/MapPage/RegionInfo.tsx

@@ -1,16 +1,17 @@
-import React from 'react'
+import React, { useState, useCallback } from "react";
-import {createPortal} from 'react-dom'
+import { createPortal } from "react-dom";
-import {useTranslation} from 'react-i18next'
+import _ from "lodash";
-import {List, Header, Icon, Button} from 'semantic-ui-react'
+import { List, Header, Icon, Button } from "semantic-ui-react";
+import { useTranslation } from "react-i18next";
 
-import styles from './styles.module.less'
+import styles from "./styles.module.less";
 
 export default function RegionInfo({ region, mapInfoPortal, onClose }) {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
   const content = (
     <>
       <div className={styles.closeHeader}>
-        <Header as="h3">{region.properties.name || t('MapPage.regionInfo.unnamedRegion')}</Header>
+        <Header as="h3">{region.properties.name || t(`MapPage.regionInfo.unnamedRegion`)}</Header>
         <Button primary icon onClick={onClose}>
           <Icon name="close" />
         </Button>
@@ -18,14 +19,17 @@ export default function RegionInfo({region, mapInfoPortal, onClose}) {
 
       <List>
         <List.Item>
-          <List.Header>{t('MapPage.regionInfo.eventCount')}</List.Header>
+          <List.Header>{t(`MapPage.regionInfo.eventNumber`)}</List.Header>
           <List.Content>{region.properties.overtaking_event_count ?? 0}</List.Content>
         </List.Item>
       </List>
     </>
-  )
+  );
 
   return content && mapInfoPortal
-    ? createPortal(<div className={styles.mapInfoBox}>{content}</div>, mapInfoPortal)
+    ? createPortal(
-    : null
+        <div className={styles.mapInfoBox}>{content}</div>,
+        mapInfoPortal
+      )
+    : null;
 }

frontend/src/pages/MapPage/RoadInfo.tsx

@@ -1,57 +1,74 @@
-import React, {useState, useCallback} from 'react'
+import React, { useState, useCallback } from "react";
-import {createPortal} from 'react-dom'
+import _ from "lodash";
-import _ from 'lodash'
+import {
-import {Segment, Menu, Header, Label, Icon, Table, Message, Button} from 'semantic-ui-react'
+  Segment,
-import {Layer, Source} from 'react-map-gl'
+  Menu,
-import {of, from, concat} from 'rxjs'
+  Header,
-import {useObservable} from 'rxjs-hooks'
+  Label,
-import {switchMap, distinctUntilChanged} from 'rxjs/operators'
+  Icon,
-import {Chart} from 'components'
+  Table,
-import {pairwise} from 'utils'
+  Message,
-import {useTranslation} from 'react-i18next'
+  Button,
+} from "semantic-ui-react";
+import { Layer, Source } from "react-map-gl";
+import { of, from, concat } from "rxjs";
+import { useObservable } from "rxjs-hooks";
+import { switchMap, distinctUntilChanged } from "rxjs/operators";
+import { Chart } from "components";
+import { pairwise } from "utils";
+import { useTranslation } from "react-i18next";
 
-import type {Location} from 'types'
+import type { Location } from "types";
-import api from 'api'
+import api from "api";
-import {colorByDistance, borderByZone} from 'mapstyles'
+import { colorByDistance, borderByZone } from "mapstyles";
 
-import styles from './styles.module.less'
+import styles from "./styles.module.less";
 
 function selectFromColorMap(colormap, value) {
-  let last = null
+  let last = null;
   for (let i = 0; i < colormap.length; i += 2) {
     if (colormap[i + 1] > value) {
-      return colormap[i]
+      return colormap[i];
     }
   }
-  return colormap[colormap.length - 1]
+  return colormap[colormap.length - 1];
 }
 
 const UNITS = {
-  distanceOvertaker: 'm',
+  distanceOvertaker: "m",
-  distanceStationary: 'm',
+  distanceStationary: "m",
-  speed: 'km/h',
+  speed: "km/h",
-}
+};
-const ZONE_COLORS = {urban: 'blue', rural: 'cyan', motorway: 'purple'}
+const ZONE_COLORS = { urban: "blue", rural: "cyan", motorway: "purple" };
-const CARDINAL_DIRECTIONS = ['north', 'northEast', 'east', 'southEast', 'south', 'southWest', 'west', 'northWest']
+const CARDINAL_DIRECTIONS = [
+  "north",
+  "northEast",
+  "east",
+  "southEast",
+  "south",
+  "southWest",
+  "west",
+  "northWest",
+];
 const getCardinalDirection = (t, bearing) => {
   if (bearing == null) {
-    return t('MapPage.roadInfo.cardinalDirections.unknown')
+    return t("MapPage.roadInfo.cardinalDirections.unknown");
   } else {
-    const n = CARDINAL_DIRECTIONS.length
+    const n = CARDINAL_DIRECTIONS.length;
-    const i = Math.floor(((bearing / 360.0) * n + 0.5) % n)
+    const i = Math.floor(((bearing / 360.0) * n + 0.5) % n);
-    const name = CARDINAL_DIRECTIONS[i]
+    const name = CARDINAL_DIRECTIONS[i];
-    return t(`MapPage.roadInfo.cardinalDirections.${name}`)
+    return t(`MapPage.roadInfo.cardinalDirections.${name}`);
-  }
   }
+};
 
 function RoadStatsTable({ data }) {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
   return (
     <Table size="small" compact>
       <Table.Header>
         <Table.Row>
           <Table.HeaderCell textAlign="right"></Table.HeaderCell>
-          {['distanceOvertaker', 'distanceStationary', 'speed'].map((prop) => (
+          {["distanceOvertaker", "distanceStationary", "speed"].map((prop) => (
             <Table.HeaderCell key={prop} textAlign="right">
               {t(`MapPage.roadInfo.${prop}`)}
             </Table.HeaderCell>
@@ -59,36 +76,42 @@ function RoadStatsTable({data}) {
         </Table.Row>
       </Table.Header>
       <Table.Body>
-        {['count', 'min', 'median', 'max', 'mean'].map((stat) => (
+        {["count", "min", "median", "max", "mean"].map((stat) => (
           <Table.Row key={stat}>
             <Table.Cell> {t(`MapPage.roadInfo.${stat}`)}</Table.Cell>
-            {['distanceOvertaker', 'distanceStationary', 'speed'].map((prop) => (
+            {["distanceOvertaker", "distanceStationary", "speed"].map(
+              (prop) => (
                 <Table.Cell key={prop} textAlign="right">
-                {(data[prop]?.statistics?.[stat] * (prop === `speed` && stat != 'count' ? 3.6 : 1)).toFixed(
+                  {(
-                  stat === 'count' ? 0 : 2
+                    data[prop]?.statistics?.[stat] *
-                )}
+                    (prop === `speed` && stat != "count" ? 3.6 : 1)
-                {stat !== 'count' && ` ${UNITS[prop]}`}
+                  ).toFixed(stat === "count" ? 0 : 2)}
+                  {stat !== "count" && ` ${UNITS[prop]}`}
                 </Table.Cell>
-            ))}
+              )
+            )}
           </Table.Row>
         ))}
       </Table.Body>
     </Table>
-  )
+  );
 }
 
 function HistogramChart({ bins, counts, zone }) {
-  const diff = bins[1] - bins[0]
+  const diff = bins[1] - bins[0];
-  const colortype = zone === 'rural' ? 3 : 5
+  const colortype = zone === "rural" ? 3 : 5;
   const data = _.zip(
     bins.slice(0, bins.length - 1).map((v) => v + diff / 2),
     counts
   ).map((value) => ({
     value,
     itemStyle: {
-      color: selectFromColorMap(colorByDistance()[3][colortype].slice(2), value[0]),
+      color: selectFromColorMap(
+        colorByDistance()[3][colortype].slice(2),
+        value[0]
+      ),
     },
-  }))
+  }));
 
   return (
     <Chart
@@ -96,7 +119,7 @@ function HistogramChart({bins, counts, zone}) {
       option={{
         grid: { top: 30, bottom: 30, right: 30, left: 30 },
         xAxis: {
-          type: 'value',
+          type: "value",
           axisLabel: { formatter: (v) => `${Math.round(v * 100)} cm` },
           min: 0,
           max: 2.5,
@@ -104,7 +127,7 @@ function HistogramChart({bins, counts, zone}) {
         yAxis: {},
         series: [
           {
-            type: 'bar',
+            type: "bar",
             data,
 
             barMaxWidth: 20,
@@ -112,83 +135,92 @@ function HistogramChart({bins, counts, zone}) {
         ],
       }}
     />
-  )
+  );
-}
-
-interface ArrayStats {
-  statistics: {
-    count: number
-    mean: number
-    min: number
-    max: number
-    median: number
-  }
-  histogram: {
-    bins: number[]
-    counts: number[]
-  }
-  values: number[]
-}
-
-export interface RoadDirectionInfo {
-  bearing: number
-  distanceOvertaker: ArrayStats
-  distanceStationary: ArrayStats
-  speed: ArrayStats
-}
-
-export interface RoadInfoType {
-  road: {
-    way_id: number
-    zone: 'urban' | 'rural' | null
-    name: string
-    directionality: -1 | 0 | 1
-    oneway: boolean
-    geometry: Object
-  }
-  forwards: RoadDirectionInfo
-  backwards: RoadDirectionInfo
 }
 
 export default function RoadInfo({
-  roadInfo: info,
+  clickLocation,
   hasFilters,
   onClose,
-  mapInfoPortal,
 }: {
-  roadInfo: RoadInfoType
+  clickLocation: Location | null;
-  hasFilters: boolean
+  hasFilters: boolean;
-  onClose: () => void
+  onClose: () => void;
-  mapInfoPortal: HTMLElement
 }) {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
-  const [direction, setDirection] = useState('forwards')
+  const [direction, setDirection] = useState("forwards");
 
   const onClickDirection = useCallback(
     (e, { name }) => {
-      e.preventDefault()
+      e.preventDefault();
-      e.stopPropagation()
+      e.stopPropagation();
-      setDirection(name)
+      setDirection(name);
     },
     [setDirection]
+  );
+
+  const info = useObservable(
+    (_$, inputs$) =>
+      inputs$.pipe(
+        distinctUntilChanged(_.isEqual),
+        switchMap(([location]) =>
+          location
+            ? concat(
+                of(null),
+                from(
+                  api.get("/mapdetails/road", {
+                    query: {
+                      ...location,
+                      radius: 100,
+                    },
+                  })
                 )
+              )
+            : of(null)
+        )
+      ),
+    null,
+    [clickLocation]
+  );
 
-  // TODO: change based on left-hand/right-hand traffic
+  if (!clickLocation) {
-  const offsetDirection = info.road.oneway ? 0 : direction === 'forwards' ? 1 : -1
+    return null;
+  }
 
-  const content = (
+  const loading = info == null;
+
+  const offsetDirection = info?.road?.oneway
+    ? 0
+    : direction === "forwards"
+    ? 1
+    : -1; // TODO: change based on left-hand/right-hand traffic
+
+  const content =
+    !loading && !info.road ? (
+      "No road found."
+    ) : (
       <>
-      <div className={styles.closeHeader}>
+        <Header as="h3">
-        <Header as="h3">{info?.road.name || t('MapPage.roadInfo.unnamedWay')}</Header>
+          {loading
-        <Button primary icon onClick={onClose}>
+            ? "..."
-          <Icon name="close" />
+            : info?.road.name || t("MapPage.roadInfo.unnamedWay")}
-        </Button>
+
-      </div>
+          <Button
+            style={{ float: "right" }}
+            onClick={onClose}
+            title={t("MapPage.roadInfo.closeTooltip")}
+            size="small"
+            icon="close"
+            basic
+          />
+        </Header>
 
         {hasFilters && (
           <Message info icon>
             <Icon name="info circle" small />
-          <Message.Content>{t('MapPage.roadInfo.hintFiltersNotApplied')}</Message.Content>
+            <Message.Content>
+              {t("MapPage.roadInfo.hintFiltersNotApplied")}
+            </Message.Content>
           </Message>
         )}
 
@@ -200,17 +232,26 @@ export default function RoadInfo({
 
         {info?.road.oneway && (
           <Label size="small" color="blue">
-          <Icon name="long arrow alternate right" fitted /> {t('MapPage.roadInfo.oneway')}
+            <Icon name="long arrow alternate right" fitted />{" "}
+            {t("MapPage.roadInfo.oneway")}
           </Label>
         )}
 
         {info?.road.oneway ? null : (
-        <Menu size="tiny" pointing>
+          <Menu size="tiny" fluid secondary>
-          <Menu.Item header>{t('MapPage.roadInfo.direction')}</Menu.Item>
+            <Menu.Item header>{t("MapPage.roadInfo.direction")}</Menu.Item>
-          <Menu.Item name="forwards" active={direction === 'forwards'} onClick={onClickDirection}>
+            <Menu.Item
+              name="forwards"
+              active={direction === "forwards"}
+              onClick={onClickDirection}
+            >
               {getCardinalDirection(t, info?.forwards?.bearing)}
             </Menu.Item>
-          <Menu.Item name="backwards" active={direction === 'backwards'} onClick={onClickDirection}>
+            <Menu.Item
+              name="backwards"
+              active={direction === "backwards"}
+              onClick={onClickDirection}
+            >
               {getCardinalDirection(t, info?.backwards?.bearing)}
             </Menu.Item>
           </Menu>
@@ -220,12 +261,16 @@ export default function RoadInfo({
 
         {info?.[direction]?.distanceOvertaker?.histogram && (
           <>
-          <Header as="h5">{t('MapPage.roadInfo.overtakerDistanceDistribution')}</Header>
+            <Header as="h5">
-          <HistogramChart {...info[direction]?.distanceOvertaker?.histogram} />
+              {t("MapPage.roadInfo.overtakerDistanceDistribution")}
+            </Header>
+            <HistogramChart
+              {...info[direction]?.distanceOvertaker?.histogram}
+            />
           </>
         )}
       </>
-  )
+    );
 
   return (
     <>
@@ -235,14 +280,22 @@ export default function RoadInfo({
             id="route"
             type="line"
             paint={{
-              'line-width': ['interpolate', ['linear'], ['zoom'], 14, 6, 17, 12],
+              "line-width": [
-              'line-color': '#18FFFF',
+                "interpolate",
-              'line-opacity': 0.5,
+                ["linear"],
+                ["zoom"],
+                14,
+                6,
+                17,
+                12,
+              ],
+              "line-color": "#18FFFF",
+              "line-opacity": 0.5,
               ...{
-                'line-offset': [
+                "line-offset": [
-                  'interpolate',
+                  "interpolate",
-                  ['exponential', 1.5],
+                  ["exponential", 1.5],
-                  ['zoom'],
+                  ["zoom"],
                   12,
                   offsetDirection,
                   19,
@@ -254,7 +307,11 @@ export default function RoadInfo({
         </Source>
       )}
 
-      {content && mapInfoPortal && createPortal(<div className={styles.mapInfoBox}>{content}</div>, mapInfoPortal)}
+      {content && (
+        <div className={styles.mapInfoBox}>
+          <Segment loading={loading}>{content}</Segment>
+        </div>
+      )}
     </>
-  )
+  );
 }

frontend/src/pages/MapPage/index.tsx

@@ -1,230 +1,284 @@
-import React, {useState, useCallback, useMemo, useRef} from 'react'
+import React, { useState, useCallback, useMemo, useRef } from "react";
-import _ from 'lodash'
+import _ from "lodash";
-import {connect} from 'react-redux'
+import { connect } from "react-redux";
-import {Button} from 'semantic-ui-react'
+import { Button } from "semantic-ui-react";
-import {Layer, Source} from 'react-map-gl'
+import { Layer, Source } from "react-map-gl";
-import produce from 'immer'
+import produce from "immer";
-import classNames from 'classnames'
+import classNames from "classnames";
 
-import api from 'api'
+import api from "api";
-import type {Location} from 'types'
+import type { Location } from "types";
-import {Page, Map} from 'components'
+import { Page, Map } from "components";
-import {useConfig} from 'config'
+import { useConfig } from "config";
-import {colorByDistance, colorByCount, getRegionLayers, borderByZone, isValidAttribute} from 'mapstyles'
+import {
-import {useMapConfig} from 'reducers/mapConfig'
+  colorByDistance,
+  colorByCount,
+  borderByZone,
+  reds,
+  isValidAttribute,
+  getRegionLayers
+} from "mapstyles";
+import { useMapConfig } from "reducers/mapConfig";
 
-import RoadInfo, {RoadInfoType} from './RoadInfo'
+import RoadInfo from "./RoadInfo";
-import RegionInfo from './RegionInfo'
+import RegionInfo from "./RegionInfo";
-import LayerSidebar from './LayerSidebar'
+import LayerSidebar from "./LayerSidebar";
-import styles from './styles.module.less'
+import styles from "./styles.module.less";
 
 const untaggedRoadsLayer = {
-  id: 'obs_roads_untagged',
+  id: "obs_roads_untagged",
-  type: 'line',
+  type: "line",
-  source: 'obs',
+  source: "obs",
-  'source-layer': 'obs_roads',
+  "source-layer": "obs_roads",
   minzoom: 12,
-  filter: ['!', ['to-boolean', ['get', 'distance_overtaker_mean']]],
+  filter: ["!", ["to-boolean", ["get", "distance_overtaker_mean"]]],
   layout: {
-    'line-cap': 'round',
+    "line-cap": "round",
-    'line-join': 'round',
+    "line-join": "round",
   },
   paint: {
-    'line-width': ['interpolate', ['exponential', 1.5], ['zoom'], 12, 2, 17, 2],
+    "line-width": ["interpolate", ["exponential", 1.5], ["zoom"], 12, 2, 17, 2],
-    'line-color': '#ABC',
+    "line-color": "#ABC",
-    // "line-opacity": ["interpolate", ["linear"], ["zoom"], 14, 0, 15, 1],
+    "line-opacity": ["interpolate", ["linear"], ["zoom"], 14, 0, 15, 1],
-    'line-offset': [
+    "line-offset": [
-      'interpolate',
+      "interpolate",
-      ['exponential', 1.5],
+      ["exponential", 1.5],
-      ['zoom'],
+      ["zoom"],
       12,
-      ['get', 'offset_direction'],
+      ["get", "offset_direction"],
       19,
-      ['*', ['get', 'offset_direction'], 8],
+      ["*", ["get", "offset_direction"], 8],
     ],
   },
-}
+};
 
-const getUntaggedRoadsLayer = (colorAttribute) =>
+const getUntaggedRoadsLayer = (colorAttribute, maxCount) =>
   produce(untaggedRoadsLayer, (draft) => {
-    draft.filter = ['!', isValidAttribute(colorAttribute)]
+    draft.filter = ["!", isValidAttribute(colorAttribute)];
-  })
+  });
 
 const getRoadsLayer = (colorAttribute, maxCount) =>
   produce(untaggedRoadsLayer, (draft) => {
-    draft.id = 'obs_roads_normal'
+    draft.id = "obs_roads_normal";
     draft.filter = isValidAttribute(colorAttribute)
     draft.minzoom = 10
-    draft.paint['line-width'][6] = 6 // scale bigger on zoom
+    draft.paint["line-width"][6] = 6; // scale bigger on zoom
-    draft.paint['line-color'] = colorAttribute.startsWith('distance_')
+    draft.paint["line-color"] = colorAttribute.startsWith("distance_")
       ? colorByDistance(colorAttribute)
-      : colorAttribute.endsWith('_count')
+      : colorAttribute.endsWith("_count")
       ? colorByCount(colorAttribute, maxCount)
-      : colorAttribute.endsWith('zone')
+      : colorAttribute.endsWith("zone")
       ? borderByZone()
-      : '#DDD'
+      : "#DDD";
-    // draft.paint["line-opacity"][3] = 12;
+    draft.paint["line-opacity"][3] = 10;
-    // draft.paint["line-opacity"][5] = 13;
+    draft.paint["line-opacity"][5] = 11;
-  })
+  });
 
 const getEventsLayer = () => ({
-  id: 'obs_events',
+  id: "obs_events",
-  type: 'circle',
+  type: "circle",
-  source: 'obs',
+  source: "obs",
-  'source-layer': 'obs_events',
+  "source-layer": "obs_events",
   paint: {
-    'circle-radius': ['interpolate', ['linear'], ['zoom'], 14, 3, 17, 8],
+    "circle-radius": ["interpolate", ["linear"], ["zoom"], 14, 3, 17, 8],
-    'circle-opacity': ['interpolate', ['linear'], ['zoom'], 8, 0.1, 9, 0.3, 10, 0.5, 11, 1],
+    "circle-color": colorByDistance("distance_overtaker"),
-    'circle-color': colorByDistance('distance_overtaker'),
   },
-  minzoom: 8,
+  minzoom: 11,
-})
+});
 
 const getEventsTextLayer = () => ({
-  id: 'obs_events_text',
+  id: "obs_events_text",
-  type: 'symbol',
+  type: "symbol",
   minzoom: 18,
-  source: 'obs',
+  source: "obs",
-  'source-layer': 'obs_events',
+  "source-layer": "obs_events",
   layout: {
-    'text-field': [
+    "text-field": [
-      'number-format',
+      "number-format",
-      ['get', 'distance_overtaker'],
+      ["get", "distance_overtaker"],
-      {'min-fraction-digits': 2, 'max-fraction-digits': 2},
+      { "min-fraction-digits": 2, "max-fraction-digits": 2 },
     ],
-    'text-allow-overlap': true,
+    "text-allow-overlap": true,
-    'text-size': 14,
+    "text-font": ["Open Sans Bold", "Arial Unicode MS Regular"],
-    'text-keep-upright': false,
+    "text-size": 14,
-    'text-anchor': 'left',
+    "text-keep-upright": false,
-    'text-radial-offset': 1,
+    "text-anchor": "left",
-    'text-rotate': ['-', 90, ['*', ['get', 'course'], 180 / Math.PI]],
+    "text-radial-offset": 1,
-    'text-rotation-alignment': 'map',
+    "text-rotate": ["-", 90, ["*", ["get", "course"], 180 / Math.PI]],
+    "text-rotation-alignment": "map",
   },
   paint: {
-    'text-halo-color': 'rgba(255, 255, 255, 1)',
+    "text-halo-color": "rgba(255, 255, 255, 1)",
-    'text-halo-width': 1,
+    "text-halo-width": 1,
-    'text-opacity': ['interpolate', ['linear'], ['zoom'], 15, 0, 15.3, 1],
+    "text-opacity": ["interpolate", ["linear"], ["zoom"], 15, 0, 15.3, 1],
   },
-})
+});
 
 interface RegionInfo {
   properties: {
-    admin_level: number
+    admin_level: number;
-    name: string
+    name: string;
-    overtaking_event_count: number
+    overtaking_event_count: number;
+  };
 }
+interface ArrayStats {
+  statistics: {
+    count: number;
+    mean: number;
+    min: number;
+    max: number;
+    median: number;
+  };
+  histogram: {
+    bins: number[];
+    counts: number[];
+  };
+  values: number[];
+}
+interface RoadDirectionInfo {
+  bearing: number;
+  distanceOvertaker: ArrayStats;
+  distanceStationary: ArrayStats;
+  speed: ArrayStats;
 }
 
-type Details = {type: 'road'; road: RoadInfoType} | {type: 'region'; region: RegionInfo}
+interface RoadInfo {
+  road: {
+    way_id: number;
+    zone: "urban" | "rural" | null;
+    name: string;
+    directionality: -1 | 0 | 1;
+    oneway: boolean;
+    geometry: Object;
+  };
+  forwards: RoadDirectionInfo;
+  backwards: RoadDirectionInfo;
+}
+type Details =
+  | { type: "road"; road: Object }
+  | { type: "region"; region: RegionInfo };
+
 
 function MapPage({ login }) {
-  const {obsMapSource, banner} = useConfig() || {}
+  const { obsMapSource , banner } = useConfig() || {};
-  const [details, setDetails] = useState<null | Details>(null)
+  const [details, setDetails] = useState<null | Details>(null);
 
-  const onCloseDetails = useCallback(() => setDetails(null), [setDetails])
+  const [clickLocation, setClickLocation] = useState<{longitude: number; latitude: number} | null>(null)
 
-  const mapConfig = useMapConfig()
+  const onCloseDetails = useCallback(() => setDetails(null), [setDetails]);
 
-  const viewportRef = useRef()
+  const mapConfig = useMapConfig();
-  const mapInfoPortal = useRef()
+
+  const viewportRef = useRef();
+  const mapInfoPortal = useRef();
 
   const onViewportChange = useCallback(
     (viewport) => {
-      viewportRef.current = viewport
+      viewportRef.current = viewport;
     },
     [viewportRef]
-  )
+  );
 
   const onClick = useCallback(
     async (e) => {
       // check if we clicked inside the mapInfoBox, if so, early exit
-      let node = e.target
+      let node = e.target;
       while (node) {
-        if ([styles.mapInfoBox, styles.mapToolbar].some((className) => node?.classList?.contains(className))) {
+        if (
-          return
+          [styles.mapInfoBox, styles.mapToolbar].some((className) =>
-        }
+            node?.classList?.contains(className)
-        node = node.parentNode
-      }
-
-      const {zoom} = viewportRef.current
-
-      if (zoom < 10) {
-        const clickedRegion = e.features?.find((f) => f.source === 'obs' && f.sourceLayer === 'obs_regions')
-        setDetails(clickedRegion ? {type: 'region', region: clickedRegion} : null)
-      } else {
-        const road = await api.get('/mapdetails/road', {
-          query: {
-            longitude: e.lngLat[0],
-            latitude: e.lngLat[1],
-            radius: 100,
-          },
-        })
-        setDetails(road?.road ? {type: 'road', road} : null)
-      }
-    },
-    [setDetails]
           )
+        ) {
+          return;
+        }
+        node = node.parentNode;
+      }
 
-  const [layerSidebar, setLayerSidebar] = useState(true)
+      const { zoom } = viewportRef.current;
+
+      if (zoom < 11) {
+        const clickedRegion = e.features?.find(
+          (f) => f.source === "obs" && f.sourceLayer === "obs_regions"
+        );
+        setDetails(
+          clickedRegion ? { type: "region", region: clickedRegion } : null
+        );
+      } else {
+              setClickLocation({longitude: e.lngLat[0], latitude: e.lngLat[1]})
+       }
+     },
+    [setClickLocation]
+  );
+  const onCloseRoadInfo = useCallback(() => {
+    setClickLocation(null);
+  }, [setClickLocation]);
+
+  const [layerSidebar, setLayerSidebar] = useState(true);
 
   const {
     obsRoads: { attribute, maxCount },
-  } = mapConfig
+  } = mapConfig;
 
-  const layers = []
+  const layers = [];
 
-  const untaggedRoadsLayerCustom = useMemo(() => getUntaggedRoadsLayer(attribute), [attribute])
+  const untaggedRoadsLayerCustom = useMemo(
+    () => getUntaggedRoadsLayer(attribute),
+    [attribute]
+  );
   if (mapConfig.obsRoads.show && mapConfig.obsRoads.showUntagged) {
     layers.push(untaggedRoadsLayerCustom)
   }
 
-  const roadsLayer = useMemo(() => getRoadsLayer(attribute, maxCount), [attribute, maxCount])
+  const roadsLayer = useMemo(
+    () => getRoadsLayer(attribute, maxCount),
+    [attribute, maxCount]
+  );
   if (mapConfig.obsRoads.show) {
-    layers.push(roadsLayer)
+    layers.push(roadsLayer);
   }
 
-  const regionLayers = useMemo(() => getRegionLayers(), [])
+  const regionLayers = useMemo(() => getRegionLayers(), []);
   if (mapConfig.obsRegions.show) {
-    layers.push(...regionLayers)
+    layers.push(...regionLayers);
   }
 
   const eventsLayer = useMemo(() => getEventsLayer(), [])
   const eventsTextLayer = useMemo(() => getEventsTextLayer(), [])
-
   if (mapConfig.obsEvents.show) {
-    layers.push(eventsLayer)
+    layers.push(eventsLayer);
-    layers.push(eventsTextLayer)
+    layers.push(eventsTextLayer);
   }
 
   const onToggleLayerSidebarButtonClick = useCallback(
     (e) => {
-      e.stopPropagation()
+      e.stopPropagation();
-      e.preventDefault()
+      e.preventDefault();
-      console.log('toggl;e')
+      console.log("toggl;e");
-      setLayerSidebar((v) => !v)
+      setLayerSidebar((v) => !v);
     },
     [setLayerSidebar]
-  )
+  );
 
   if (!obsMapSource) {
-    return null
+    return null;
   }
 
-  const tiles = obsMapSource?.tiles?.map((tileUrl: string) => {
+  const tiles = obsMapSource?.tiles?.map(
+    (tileUrl: string) => {
       const query = new URLSearchParams()
       if (login) {
         if (mapConfig.filters.currentUser) {
-        query.append('user', login.id)
+          query.append('user', login.username)
         }
 
-      if (mapConfig.filters.dateMode === 'range') {
+        if (mapConfig.filters.dateMode === "range") {
           if (mapConfig.filters.startDate) {
             query.append('start', mapConfig.filters.startDate)
           }
           if (mapConfig.filters.endDate) {
             query.append('end', mapConfig.filters.endDate)
           }
-      } else if (mapConfig.filters.dateMode === 'threshold') {
+        } else if (mapConfig.filters.dateMode === "threshold") {
           if (mapConfig.filters.startDate) {
             query.append(mapConfig.filters.thresholdAfter ? 'start' : 'end', mapConfig.filters.startDate)
           }
@@ -232,22 +286,41 @@ function MapPage({login}) {
       }
       const queryString = String(query)
       return tileUrl + (queryString  ? '?' : '') + queryString
-  })
+    }
+  );
 
-  const hasFilters: boolean = login && (mapConfig.filters.currentUser || mapConfig.filters.dateMode !== 'none')
+  const hasFilters: boolean =
+    login &&
+    (mapConfig.filters.currentUser || mapConfig.filters.dateMode !== "none");
 
   return (
     <Page fullScreen title="Map">
-      <div className={classNames(styles.mapContainer, banner ? styles.hasBanner : null)} ref={mapInfoPortal}>
+      <div
+        className={classNames(
+          styles.mapContainer,
+          banner ? styles.hasBanner : null
+        )}
+        ref={mapInfoPortal}
+      >
         {layerSidebar && (
           <div className={styles.mapSidebar}>
             <LayerSidebar />
           </div>
         )}
         <div className={styles.map}>
-          <Map viewportFromUrl onClick={onClick} hasToolbar onViewportChange={onViewportChange}>
+          <Map viewportFromUrl onClick={onClick} onViewportChange={onViewportChange} hasToolbar>
             <div className={styles.mapToolbar}>
-              <Button primary icon="bars" active={layerSidebar} onClick={onToggleLayerSidebarButtonClick} />
+              <Button
+              style={{
+                position: "absolute",
+                left: 16,
+                top: 16,
+              }}
+                primary
+                icon="bars"
+                active={layerSidebar}
+                onClick={onToggleLayerSidebarButtonClick}
+              />
             </div>
             <Source id="obs" {...obsMapSource} tiles={tiles}>
               {layers.map((layer) => (
@@ -255,23 +328,24 @@ function MapPage({login}) {
               ))}
             </Source>
 
-            {details?.type === 'road' && details?.road?.road && (
+
             <RoadInfo
-                roadInfo={details.road}
+              {...{ clickLocation, hasFilters, onClose: onCloseRoadInfo }}
+            />
+
+
+            {details?.type === "region" && details?.region && (
+              <RegionInfo
+                region={details.region}
                 mapInfoPortal={mapInfoPortal.current}
                 onClose={onCloseDetails}
-                {...{hasFilters}}
               />
             )}
-
-            {details?.type === 'region' && details?.region && (
-              <RegionInfo region={details.region} mapInfoPortal={mapInfoPortal.current} onClose={onCloseDetails} />
-            )}
           </Map>
         </div>
       </div>
     </Page>
-  )
+  );
 }
 
-export default connect((state) => ({login: state.login}))(MapPage)
+export default connect((state) => ({ login: state.login }))(MapPage);

frontend/src/pages/MapPage/styles.module.less

@@ -17,33 +17,28 @@
   background: white;
   border-right: 1px solid @borderColor;
   padding: 1rem;
-  overflow-y: auto;
 }
 
 .map {
   flex: 1 1 0;
-  overflow: hidden;
 }
 
 .mapInfoBox {
+  position: absolute;
+  right: 16px;
+  top: 32px;
+  max-height: 100%;
   width: 36rem;
   overflow: auto;
-  border-left: 1px solid @borderColor;
-  background: white;
-  padding: 16px;
-}
-
-.copyright {
-  color: #888;
-  font-size: 0.8em;
-  line-height: 1.4;
-  margin-block-start: 1em;
 }
 
 .mapToolbar {
   position: absolute;
   left: 16px;
   top: 16px;
+  border-left: 1px solid @borderColor;
+  background: white;
+  padding: 16px;
 }
 
 .closeHeader {
@@ -56,10 +51,7 @@
 @media @mobile {
   .mapContainer {
     height: auto;
-    min-height: calc(100vh - @menuHeightMobile);
+    min-height: calc(100vh - @menuHeight);
-    &.hasBanner {
-      height: calc(100vh - @menuHeightMobile - 50px);
-    }
     flex-direction: column;
   }
 

frontend/src/pages/MyTracksPage.tsx

@@ -1,360 +0,0 @@
-import React, {useCallback, useMemo, useState} from 'react'
-import {connect} from 'react-redux'
-import {
-  Accordion,
-  Button,
-  Checkbox,
-  Confirm,
-  Header,
-  Icon,
-  Item,
-  List,
-  Loader,
-  Dropdown,
-  SemanticCOLORS,
-  SemanticICONS,
-  Table,
-} from 'semantic-ui-react'
-import {useObservable} from 'rxjs-hooks'
-import {Link} from 'react-router-dom'
-import {of, from, concat, BehaviorSubject, combineLatest} from 'rxjs'
-import {map, switchMap, distinctUntilChanged} from 'rxjs/operators'
-import _ from 'lodash'
-import {useTranslation} from 'react-i18next'
-
-import type {ProcessingStatus, Track, UserDevice} from 'types'
-import {Page, FormattedDate, Visibility} from 'components'
-import api from 'api'
-import {useCallbackRef, formatDistance, formatDuration} from 'utils'
-
-import download from 'downloadjs'
-
-const COLOR_BY_STATUS: Record<ProcessingStatus, SemanticCOLORS> = {
-  error: 'red',
-  complete: 'green',
-  created: 'grey',
-  queued: 'orange',
-  processing: 'orange',
-}
-
-const ICON_BY_STATUS: Record<ProcessingStatus, SemanticICONS> = {
-  error: 'warning sign',
-  complete: 'check circle outline',
-  created: 'bolt',
-  queued: 'bolt',
-  processing: 'bolt',
-}
-
-function ProcessingStatusLabel({status}: {status: ProcessingStatus}) {
-  const {t} = useTranslation()
-  return (
-    <span title={t(`TracksPage.processing.${status}`)}>
-      <Icon color={COLOR_BY_STATUS[status]} name={ICON_BY_STATUS[status]} />
-    </span>
-  )
-}
-
-function SortableHeader({children, setOrderBy, orderBy, reversed, setReversed, name, ...props}) {
-  const toggleSort = (e) => {
-    e.preventDefault()
-    e.stopPropagation()
-
-    if (orderBy === name) {
-      if (!reversed) {
-        setReversed(true)
-      } else {
-        setReversed(false)
-        setOrderBy(null)
-      }
-    } else {
-      setReversed(false)
-      setOrderBy(name)
-    }
-  }
-
-  let icon = orderBy === name ? (reversed ? 'sort descending' : 'sort ascending') : null
-
-  return (
-    <Table.HeaderCell {...props}>
-      <div onClick={toggleSort}>
-        {children}
-        <Icon name={icon} />
-      </div>
-    </Table.HeaderCell>
-  )
-}
-
-type Filters = {
-  userDeviceId?: null | number
-  visibility?: null | boolean
-}
-
-function TrackFilters({
-  filters,
-  setFilters,
-  deviceNames,
-}: {
-  filters: Filters
-  setFilters: (f: Filters) => void
-  deviceNames: null | Record<number, string>
-}) {
-  return (
-    <List horizontal>
-      <List.Item>
-        <List.Header>Device</List.Header>
-        <Dropdown
-          selection
-          clearable
-          options={[
-            {value: 0, key: '__none__', text: 'All my devices'},
-            ..._.sortBy(Object.entries(deviceNames ?? {}), 1).map(([deviceId, deviceName]: [string, string]) => ({
-              value: Number(deviceId),
-              key: deviceId,
-              text: deviceName,
-            })),
-          ]}
-          value={filters?.userDeviceId ?? 0}
-          onChange={(_e, {value}) => setFilters({...filters, userDeviceId: (value as number) || null})}
-        />
-      </List.Item>
-
-      <List.Item>
-        <List.Header>Visibility</List.Header>
-        <Dropdown
-          selection
-          clearable
-          options={[
-            {value: 'none', key: 'any', text: 'Any'},
-            {value: true, key: 'public', text: 'Public'},
-            {value: false, key: 'private', text: 'Private'},
-          ]}
-          value={filters?.visibility ?? 'none'}
-          onChange={(_e, {value}) =>
-            setFilters({
-              ...filters,
-              visibility: value === 'none' ? null : (value as boolean),
-            })
-          }
-        />
-      </List.Item>
-    </List>
-  )
-}
-
-function TracksTable({title}) {
-  const [orderBy, setOrderBy] = useState('recordedAt')
-  const [reversed, setReversed] = useState(true)
-  const [showFilters, setShowFilters] = useState(false)
-  const [filters, setFilters] = useState<Filters>({})
-  const [selectedTracks, setSelectedTracks] = useState<Record<string, boolean>>({})
-
-  const toggleTrackSelection = useCallbackRef((slug: string, selected?: boolean) => {
-    const newSelected = selected ?? !selectedTracks[slug]
-    setSelectedTracks(_.pickBy({...selectedTracks, [slug]: newSelected}, _.identity))
-  })
-
-  const query = _.pickBy(
-    {
-      limit: 1000,
-      offset: 0,
-      order_by: orderBy,
-      reversed: reversed ? 'true' : 'false',
-      user_device_id: filters?.userDeviceId,
-      public: filters?.visibility,
-    },
-    (x) => x != null
-  )
-
-  const forceUpdate$ = useMemo(() => new BehaviorSubject(null), [])
-  const tracks: Track[] | null = useObservable(
-    (_$, inputs$) =>
-      combineLatest([
-        inputs$.pipe(
-          map(([query]) => query),
-          distinctUntilChanged(_.isEqual)
-        ),
-        forceUpdate$,
-      ]).pipe(switchMap(([query]) => concat(of(null), from(api.get('/tracks/feed', {query}).then((r) => r.tracks))))),
-    null,
-    [query]
-  )
-
-  const deviceNames: null | Record<number, string> = useObservable(() =>
-    from(api.get('/user/devices')).pipe(
-      map((response: UserDevice[]) =>
-        Object.fromEntries(response.map((device) => [device.id, device.displayName || device.identifier]))
-      )
-    )
-  )
-
-  const {t} = useTranslation()
-
-  const p = {orderBy, setOrderBy, reversed, setReversed}
-
-  const selectedCount = Object.keys(selectedTracks).length
-  const noneSelected = selectedCount === 0
-  const allSelected = selectedCount === tracks?.length
-  const selectAll = () => {
-    setSelectedTracks(Object.fromEntries(tracks?.map((t) => [t.slug, true]) ?? []))
-  }
-  const selectNone = () => {
-    setSelectedTracks({})
-  }
-
-  const bulkAction = async (action: string) => {
-    const response = await api.post('/tracks/bulk', {
-      body: {
-        action,
-        tracks: Object.keys(selectedTracks),
-      },
-      returnResponse: true,
-    })
-    if (action === 'download') {
-      const contentType = response.headers.get('content-type') ?? 'application/x-gtar'
-
-      const filename = response.headers.get('content-disposition')?.match(/filename="([^"]+)"/)?.[1] ?? 'tracks.tar.bz2'
-      download(await response.blob(), filename, contentType)
-    }
-
-    setShowBulkDelete(false)
-    setSelectedTracks({})
-    forceUpdate$.next(null)
-  }
-  const [showBulkDelete, setShowBulkDelete] = useState(false)
-
-  return (
-    <>
-      <div style={{float: 'right'}}>
-        <Dropdown disabled={noneSelected} text="Bulk actions" floating button>
-          <Dropdown.Menu>
-            <Dropdown.Header>Selection of {selectedCount} tracks</Dropdown.Header>
-            <Dropdown.Item onClick={() => bulkAction('makePrivate')}>Make private</Dropdown.Item>
-            <Dropdown.Item onClick={() => bulkAction('makePublic')}>Make public</Dropdown.Item>
-            <Dropdown.Item onClick={() => bulkAction('reprocess')}>Reprocess</Dropdown.Item>
-            <Dropdown.Item onClick={() => bulkAction('download')}>Download</Dropdown.Item>
-            <Dropdown.Item onClick={() => setShowBulkDelete(true)}>Delete</Dropdown.Item>
-          </Dropdown.Menu>
-        </Dropdown>
-        <Link component={UploadButton} to="/upload" />
-      </div>
-
-      <Header as="h1">{title}</Header>
-      <div style={{clear: 'both'}}>
-        <Loader content={t('general.loading')} active={tracks == null} />
-
-        <Accordion>
-          <Accordion.Title active={showFilters} index={0} onClick={() => setShowFilters(!showFilters)}>
-            <Icon name="dropdown" />
-            Filters
-          </Accordion.Title>
-          <Accordion.Content active={showFilters}>
-            <TrackFilters {...{filters, setFilters, deviceNames}} />
-          </Accordion.Content>
-        </Accordion>
-
-        <Confirm
-          open={showBulkDelete}
-          onCancel={() => setShowBulkDelete(false)}
-          onConfirm={() => bulkAction('delete')}
-          content={`Are you sure you want to delete ${selectedCount} tracks?`}
-          confirmButton={t('general.delete')}
-          cancelButton={t('general.cancel')}
-        />
-
-        <Table compact>
-          <Table.Header>
-            <Table.Row>
-              <Table.HeaderCell>
-                <Checkbox
-                  checked={allSelected}
-                  indeterminate={!allSelected && !noneSelected}
-                  onClick={() => (noneSelected ? selectAll() : selectNone())}
-                />
-              </Table.HeaderCell>
-
-              <SortableHeader {...p} name="title">
-                Title
-              </SortableHeader>
-              <SortableHeader {...p} name="recordedAt">
-                Recorded at
-              </SortableHeader>
-              <SortableHeader {...p} name="visibility">
-                Visibility
-              </SortableHeader>
-              <SortableHeader {...p} name="length" textAlign="right">
-                Length
-              </SortableHeader>
-              <SortableHeader {...p} name="duration" textAlign="right">
-                Duration
-              </SortableHeader>
-              <SortableHeader {...p} name="user_device_id">
-                Device
-              </SortableHeader>
-            </Table.Row>
-          </Table.Header>
-
-          <Table.Body>
-            {tracks?.map((track: Track) => (
-              <Table.Row key={track.slug}>
-                <Table.Cell>
-                  <Checkbox
-                    onClick={(e) => toggleTrackSelection(track.slug)}
-                    checked={selectedTracks[track.slug] ?? false}
-                  />
-                </Table.Cell>
-                <Table.Cell>
-                  {track.processingStatus == null ? null : <ProcessingStatusLabel status={track.processingStatus} />}
-                  <Item.Header as={Link} to={`/tracks/${track.slug}`}>
-                    {track.title || t('general.unnamedTrack')}
-                  </Item.Header>
-                </Table.Cell>
-
-                <Table.Cell>
-                  <FormattedDate date={track.recordedAt} />
-                </Table.Cell>
-
-                <Table.Cell>{track.public == null ? null : <Visibility public={track.public} />}</Table.Cell>
-
-                <Table.Cell textAlign="right">{formatDistance(track.length)}</Table.Cell>
-
-                <Table.Cell textAlign="right">{formatDuration(track.duration)}</Table.Cell>
-
-                <Table.Cell>{track.userDeviceId ? deviceNames?.[track.userDeviceId] ?? '...' : null}</Table.Cell>
-              </Table.Row>
-            ))}
-          </Table.Body>
-        </Table>
-      </div>
-    </>
-  )
-}
-
-function UploadButton({navigate, ...props}) {
-  const {t} = useTranslation()
-  const onClick = useCallback(
-    (e) => {
-      e.preventDefault()
-      navigate()
-    },
-    [navigate]
-  )
-  return (
-    <Button onClick={onClick} {...props} color="green">
-      {t('TracksPage.upload')}
-    </Button>
-  )
-}
-
-const MyTracksPage = connect((state) => ({login: (state as any).login}))(function MyTracksPage({login}) {
-  const {t} = useTranslation()
-
-  const title = t('TracksPage.titleUser')
-
-  return (
-    <Page title={title}>
-      <TracksTable {...{title}} />
-    </Page>
-  )
-})
-
-export default MyTracksPage

frontend/src/pages/NotFoundPage.js

@@ -1,12 +1,12 @@
 import React from 'react'
 import {Button, Header} from 'semantic-ui-react'
 import {useHistory} from 'react-router-dom'
-import {useTranslation} from 'react-i18next'
+import { useTranslation } from "react-i18next";
 
 import {Page} from '../components'
 
 export default function NotFoundPage() {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
   const history = useHistory()
   return (
     <Page title={t('NotFoundPage.title')}>

frontend/src/pages/SettingsPage.tsx

@@ -0,0 +1,227 @@
+import React from "react";
+import { connect } from "react-redux";
+import {
+  Message,
+  Icon,
+  Grid,
+  Form,
+  Button,
+  TextArea,
+  Ref,
+  Input,
+  Header,
+  Divider,
+  Popup,
+} from "semantic-ui-react";
+import { useForm } from "react-hook-form";
+import Markdown from "react-markdown";
+import { useTranslation } from "react-i18next";
+
+import { setLogin } from "reducers/login";
+import { Page, Stats } from "components";
+import api from "api";
+import { findInput } from "utils";
+import { useConfig } from "config";
+
+const SettingsPage = connect((state) => ({ login: state.login }), { setLogin })(
+  function SettingsPage({ login, setLogin }) {
+    const { t } = useTranslation();
+    const { register, handleSubmit } = useForm();
+    const [loading, setLoading] = React.useState(false);
+    const [errors, setErrors] = React.useState(null);
+
+    const onSave = React.useCallback(
+      async (changes) => {
+        setLoading(true);
+        setErrors(null);
+        try {
+          const response = await api.put("/user", { body: changes });
+          setLogin(response);
+        } catch (err) {
+          setErrors(err.errors);
+        } finally {
+          setLoading(false);
+        }
+      },
+      [setLoading, setLogin, setErrors]
+    );
+
+    const onGenerateNewKey = React.useCallback(async () => {
+      setLoading(true);
+      setErrors(null);
+      try {
+        const response = await api.put("/user", {
+          body: { updateApiKey: true },
+        });
+        setLogin(response);
+      } catch (err) {
+        setErrors(err.errors);
+      } finally {
+        setLoading(false);
+      }
+    }, [setLoading, setLogin, setErrors]);
+
+    return (
+      <Page title={t("SettingsPage.title")}>
+        <Grid centered relaxed divided stackable>
+          <Grid.Row>
+            <Grid.Column width={8}>
+              <Header as="h2">{t("SettingsPage.profile.title")}</Header>
+
+              <Form onSubmit={handleSubmit(onSave)} loading={loading}>
+                <Form.Field error={errors?.username}>
+                  <label>{t("SettingsPage.profile.username.label")}</label>
+                  <Ref innerRef={findInput(register)}>
+                    <Input
+                      name="username"
+                      defaultValue={login.username}
+                      disabled
+                    />
+                  </Ref>
+                  <small>{t("SettingsPage.profile.username.hint")}</small>
+                </Form.Field>
+
+                <Message info visible>
+                  {t("SettingsPage.profile.publicNotice")}
+                </Message>
+
+                <Form.Field error={errors?.displayName}>
+                  <label>{t("SettingsPage.profile.displayName.label")}</label>
+                  <Ref innerRef={findInput(register)}>
+                    <Input
+                      name="displayName"
+                      defaultValue={login.displayName}
+                      placeholder={login.username}
+                    />
+                  </Ref>
+                  <small>
+                    {t("SettingsPage.profile.displayName.fallbackNotice")}
+                  </small>
+                </Form.Field>
+
+                <Form.Field error={errors?.bio}>
+                  <label>{t("SettingsPage.profile.bio.label")}</label>
+                  <Ref innerRef={register}>
+                    <TextArea name="bio" rows={4} defaultValue={login.bio} />
+                  </Ref>
+                </Form.Field>
+                <Form.Field error={errors?.image}>
+                  <label>{t("SettingsPage.profile.avatarUrl.label")}</label>
+                  <Ref innerRef={findInput(register)}>
+                    <Input name="image" defaultValue={login.image} />
+                  </Ref>
+                </Form.Field>
+
+                <Button type="submit" primary>
+                  {t("general.save")}
+                </Button>
+              </Form>
+            </Grid.Column>
+            <Grid.Column width={6}>
+              <ApiKeyDialog {...{ login, onGenerateNewKey }} />
+
+              <Divider />
+
+              <Stats user={login.id} />
+            </Grid.Column>
+          </Grid.Row>
+        </Grid>
+      </Page>
+    );
+  }
+);
+
+function CopyInput({ value, ...props }) {
+  const { t } = useTranslation();
+  const [success, setSuccess] = React.useState(null);
+  const onClick = async () => {
+    try {
+      await window.navigator?.clipboard?.writeText(value);
+      setSuccess(true);
+    } catch (err) {
+      setSuccess(false);
+    } finally {
+      setTimeout(() => {
+        setSuccess(null);
+      }, 2000);
+    }
+  };
+
+  return (
+    <Popup
+      trigger={
+        <Input
+          {...props}
+          value={value}
+          fluid
+          action={{ icon: "copy", onClick }}
+        />
+      }
+      position="top right"
+      open={success != null}
+      content={success ? t("general.copied") : t("general.copyError")}
+    />
+  );
+}
+
+const selectField = findInput((ref) => ref?.select());
+
+function ApiKeyDialog({ login, onGenerateNewKey }) {
+  const { t } = useTranslation();
+  const config = useConfig();
+  const [show, setShow] = React.useState(false);
+  const onClick = React.useCallback(
+    (e) => {
+      e.preventDefault();
+      setShow(true);
+    },
+    [setShow]
+  );
+
+  const onGenerateNewKeyInner = React.useCallback(
+    (e) => {
+      e.preventDefault();
+      onGenerateNewKey();
+    },
+    [onGenerateNewKey]
+  );
+
+  return (
+    <>
+      <Header as="h2">{t("SettingsPage.apiKey.title")}</Header>
+      <Markdown>{t("SettingsPage.apiKey.description")}</Markdown>
+      <div style={{ minHeight: 40, marginBottom: 16 }}>
+        {show ? (
+          login.apiKey ? (
+            <Ref innerRef={selectField}>
+              <CopyInput
+                label={t("SettingsPage.apiKey.key.label")}
+                value={login.apiKey}
+              />
+            </Ref>
+          ) : (
+            <Message warning content={t("SettingsPage.apiKey.key.empty")} />
+          )
+        ) : (
+          <Button onClick={onClick}>
+            <Icon name="lock" /> {t("SettingsPage.apiKey.key.show")}
+          </Button>
+        )}
+      </div>
+      <Markdown>{t("SettingsPage.apiKey.urlDescription")}</Markdown>
+      <div style={{ marginBottom: 16 }}>
+        <CopyInput
+          label={t("SettingsPage.apiKey.url.label")}
+          value={config?.apiUrl?.replace(/\/api$/, "") ?? "..."}
+        />
+      </div>
+      <Markdown>{t("SettingsPage.apiKey.generateDescription")}</Markdown>
+      <p></p>
+      <Button onClick={onGenerateNewKeyInner}>
+        {t("SettingsPage.apiKey.generate")}
+      </Button>
+    </>
+  );
+}
+
+export default SettingsPage;

frontend/src/pages/SettingsPage/ApiKeySettings.tsx

@@ -1,102 +0,0 @@
-import React from 'react'
-import {connect} from 'react-redux'
-import {Message, Icon, Button, Ref, Input, Segment, Popup} from 'semantic-ui-react'
-import Markdown from 'react-markdown'
-import {useTranslation} from 'react-i18next'
-
-import {setLogin} from 'reducers/login'
-import api from 'api'
-import {findInput} from 'utils'
-import {useConfig} from 'config'
-
-function CopyInput({value, ...props}) {
-  const {t} = useTranslation()
-  const [success, setSuccess] = React.useState(null)
-  const onClick = async () => {
-    try {
-      await window.navigator?.clipboard?.writeText(value)
-      setSuccess(true)
-    } catch (err) {
-      setSuccess(false)
-    } finally {
-      setTimeout(() => {
-        setSuccess(null)
-      }, 2000)
-    }
-  }
-
-  return (
-    <Popup
-      trigger={<Input {...props} value={value} fluid action={{icon: 'copy', onClick}} />}
-      position="top right"
-      open={success != null}
-      content={success ? t('general.copied') : t('general.copyError')}
-    />
-  )
-}
-
-const selectField = findInput((ref) => ref?.select())
-
-const ApiKeySettings = connect((state) => ({login: state.login}), {
-  setLogin,
-})(function ApiKeySettings({login, setLogin, setErrors}) {
-  const {t} = useTranslation()
-  const [loading, setLoading] = React.useState(false)
-  const config = useConfig()
-  const [show, setShow] = React.useState(false)
-  const onClick = React.useCallback(
-    (e) => {
-      e.preventDefault()
-      setShow(true)
-    },
-    [setShow]
-  )
-
-  const onGenerateNewKey = React.useCallback(
-    async (e) => {
-      e.preventDefault()
-      setLoading(true)
-      try {
-        const response = await api.put('/user', {
-          body: {updateApiKey: true},
-        })
-        setLogin(response)
-      } catch (err) {
-        setErrors(err.errors)
-      } finally {
-        setLoading(false)
-      }
-    },
-    [setLoading, setLogin, setErrors]
-  )
-
-  return (
-    <Segment style={{maxWidth: 600, margin: '24px auto'}}>
-      <Markdown>{t('SettingsPage.apiKey.description')}</Markdown>
-      <div style={{minHeight: 40, marginBottom: 16}}>
-        {show ? (
-          login.apiKey ? (
-            <Ref innerRef={selectField}>
-              <CopyInput label={t('SettingsPage.apiKey.key.label')} value={login.apiKey} />
-            </Ref>
-          ) : (
-            <Message warning content={t('SettingsPage.apiKey.key.empty')} />
-          )
-        ) : (
-          <Button onClick={onClick}>
-            <Icon name="lock" /> {t('SettingsPage.apiKey.key.show')}
-          </Button>
-        )}
-      </div>
-      <Markdown>{t('SettingsPage.apiKey.urlDescription')}</Markdown>
-      <div style={{marginBottom: 16}}>
-        <CopyInput label={t('SettingsPage.apiKey.url.label')} value={config?.apiUrl?.replace(/\/api$/, '') ?? '...'} />
-      </div>
-      <Markdown>{t('SettingsPage.apiKey.generateDescription')}</Markdown>
-      <p></p>
-      <Button onClick={onGenerateNewKey}>{t('SettingsPage.apiKey.generate')}</Button>
-    </Segment>
-  )
-})
-
-export default ApiKeySettings

frontend/src/pages/SettingsPage/DeviceList.tsx

@@ -1,125 +0,0 @@
-import React, {useCallback, useMemo, useRef} from 'react'
-import {useObservable} from 'rxjs-hooks'
-import {concat, from, of, Subject} from 'rxjs'
-import {Table, Button, Input} from 'semantic-ui-react'
-import {useTranslation} from 'react-i18next'
-
-import api from 'api'
-import {UserDevice} from 'types'
-import {startWith, switchMap} from 'rxjs/operators'
-
-function EditField({value, onEdit}) {
-  const [editing, setEditing] = React.useState(false)
-  const [tempValue, setTempValue] = React.useState(value)
-  const timeoutRef = useRef<null | number>(null)
-
-  const cancelTimeout = useCallback(() => {
-    if (timeoutRef.current != null) {
-      clearTimeout(timeoutRef.current)
-      timeoutRef.current = null
-    }
-  }, [timeoutRef])
-
-  const abort = useCallback(() => {
-    cancelTimeout()
-    setEditing(false)
-    setTempValue(value)
-  }, [setEditing, setTempValue, value, cancelTimeout])
-
-  const confirm = useCallback(() => {
-    console.log('confirmed')
-    cancelTimeout()
-    setEditing(false)
-    onEdit(tempValue)
-  }, [setEditing, onEdit, tempValue, cancelTimeout])
-
-  React.useEffect(() => {
-    if (value !== tempValue) {
-      setTempValue(value)
-    }
-  }, [value])
-
-  if (editing) {
-    return (
-      <>
-        <Input
-          value={tempValue}
-          onChange={(e) => setTempValue(e.target.value)}
-          onBlur={(e) => {
-            timeoutRef.current = setTimeout(abort, 20)
-          }}
-          onKeyPress={(e) => {
-            if (e.key === 'Enter') {
-              confirm()
-            } else if (e.key === 'Escape') {
-              abort()
-            }
-          }}
-          style={{marginRight: 8}}
-        />
-      </>
-    )
-  } else {
-    return (
-      <>
-        {value && <span style={{marginRight: 8}}>{value}</span>}
-        <Button icon="edit" size="tiny" onClick={() => setEditing(true)} />
-      </>
-    )
-  }
-}
-
-export default function DeviceList() {
-  const {t} = useTranslation()
-  const [loading_, setLoading] = React.useState(false)
-
-  const trigger$ = useMemo(() => new Subject(), [])
-  const devices: null | UserDevice[] = useObservable(() =>
-    trigger$.pipe(
-      startWith(null),
-      switchMap(() => concat(of(null), from(api.get('/user/devices'))))
-    )
-  )
-
-  const setDeviceDisplayName = useCallback(
-    async (deviceId: number, displayName: string) => {
-      setLoading(true)
-      try {
-        await api.put(`/user/devices/${deviceId}`, {body: {displayName}})
-      } finally {
-        setLoading(false)
-        trigger$.next(null)
-      }
-    },
-    [trigger$, setLoading]
-  )
-
-  const loading = devices == null || loading_
-
-  return (
-    <>
-      <Table compact {...{loading}}>
-        <Table.Header>
-          <Table.Row>
-            <Table.HeaderCell width={4}>{t('SettingsPage.devices.identifier')}</Table.HeaderCell>
-            <Table.HeaderCell>{t('SettingsPage.devices.alias')}</Table.HeaderCell>
-            <Table.HeaderCell />
-          </Table.Row>
-        </Table.Header>
-        <Table.Body>
-          {devices?.map((device: UserDevice) => (
-            <Table.Row key={device.id}>
-              <Table.Cell> {device.identifier}</Table.Cell>
-              <Table.Cell>
-                <EditField
-                  value={device.displayName}
-                  onEdit={(displayName: string) => setDeviceDisplayName(device.id, displayName)}
-                />
-              </Table.Cell>
-            </Table.Row>
-          ))}
-        </Table.Body>
-      </Table>
-    </>
-  )
-}

frontend/src/pages/SettingsPage/UserSettingsForm.tsx

@@ -1,77 +0,0 @@
-import React from 'react'
-import {connect} from 'react-redux'
-import {Segment, Message, Form, Button, TextArea, Ref, Input} from 'semantic-ui-react'
-import {useForm} from 'react-hook-form'
-import {useTranslation} from 'react-i18next'
-
-import {setLogin} from 'reducers/login'
-import api from 'api'
-import {findInput} from 'utils'
-
-const UserSettingsForm = connect((state) => ({login: state.login}), {
-  setLogin,
-})(function UserSettingsForm({login, setLogin, errors, setErrors}) {
-  const {t} = useTranslation()
-  const {register, handleSubmit} = useForm()
-  const [loading, setLoading] = React.useState(false)
-
-  const onSave = React.useCallback(
-    async (changes) => {
-      setLoading(true)
-      setErrors(null)
-      try {
-        const response = await api.put('/user', {body: changes})
-        setLogin(response)
-      } catch (err) {
-        setErrors(err.errors)
-      } finally {
-        setLoading(false)
-      }
-    },
-    [setLoading, setLogin, setErrors]
-  )
-
-  return (
-    <Segment style={{maxWidth: 600}}>
-      <Form onSubmit={handleSubmit(onSave)} loading={loading}>
-        <Form.Field error={errors?.username}>
-          <label>{t('SettingsPage.profile.username.label')}</label>
-          <Ref innerRef={findInput(register)}>
-            <Input name="username" defaultValue={login.username} disabled />
-          </Ref>
-          <small>{t('SettingsPage.profile.username.hint')}</small>
-        </Form.Field>
-
-        <Message info visible>
-          {t('SettingsPage.profile.publicNotice')}
-        </Message>
-
-        <Form.Field error={errors?.displayName}>
-          <label>{t('SettingsPage.profile.displayName.label')}</label>
-          <Ref innerRef={findInput(register)}>
-            <Input name="displayName" defaultValue={login.displayName} placeholder={login.username} />
-          </Ref>
-          <small>{t('SettingsPage.profile.displayName.fallbackNotice')}</small>
-        </Form.Field>
-
-        <Form.Field error={errors?.bio}>
-          <label>{t('SettingsPage.profile.bio.label')}</label>
-          <Ref innerRef={register}>
-            <TextArea name="bio" rows={4} defaultValue={login.bio} />
-          </Ref>
-        </Form.Field>
-        <Form.Field error={errors?.image}>
-          <label>{t('SettingsPage.profile.avatarUrl.label')}</label>
-          <Ref innerRef={findInput(register)}>
-            <Input name="image" defaultValue={login.image} />
-          </Ref>
-        </Form.Field>
-
-        <Button type="submit" primary>
-          {t('general.save')}
-        </Button>
-      </Form>
-    </Segment>
-  )
-})
-export default UserSettingsForm

frontend/src/pages/SettingsPage/index.tsx

@@ -1,68 +0,0 @@
-import React from 'react'
-import {connect} from 'react-redux'
-import {Header, Tab} from 'semantic-ui-react'
-import {useForm} from 'react-hook-form'
-import {useTranslation} from 'react-i18next'
-
-import {setLogin} from 'reducers/login'
-import {Page, Stats} from 'components'
-import api from 'api'
-
-import ApiKeySettings from './ApiKeySettings'
-
-import UserSettingsForm from './UserSettingsForm'
-import DeviceList from './DeviceList'
-
-const SettingsPage = connect((state) => ({login: state.login}), {setLogin})(function SettingsPage({login, setLogin}) {
-  const {t} = useTranslation()
-  const {register, handleSubmit} = useForm()
-  const [loading, setLoading] = React.useState(false)
-  const [errors, setErrors] = React.useState(null)
-
-  const onGenerateNewKey = React.useCallback(async () => {
-    setLoading(true)
-    setErrors(null)
-    try {
-      const response = await api.put('/user', {
-        body: {updateApiKey: true},
-      })
-      setLogin(response)
-    } catch (err) {
-      setErrors(err.errors)
-    } finally {
-      setLoading(false)
-    }
-  }, [setLoading, setLogin, setErrors])
-
-  return (
-    <Page title={t('SettingsPage.title')}>
-      <Header as="h1">{t('SettingsPage.title')}</Header>
-      <Tab
-        menu={{secondary: true, pointing: true}}
-        panes={[
-          {
-            menuItem: t('SettingsPage.profile.title'),
-            render: () => <UserSettingsForm {...{errors, setErrors}} />,
-          },
-
-          {
-            menuItem: t('SettingsPage.apiKey.title'),
-            render: () => <ApiKeySettings {...{errors, setErrors}} />,
-          },
-
-          {
-            menuItem: t('SettingsPage.stats.title'),
-            render: () => <Stats user={login.id} />,
-          },
-
-          {
-            menuItem: t('SettingsPage.devices.title'),
-            render: () => <DeviceList />,
-          },
-        ]}
-      />
-    </Page>
-  )
-})
-
-export default SettingsPage

frontend/src/pages/TrackEditor.tsx

@@ -1,6 +1,6 @@
-import React from 'react'
+import React from "react";
-import _ from 'lodash'
+import _ from "lodash";
-import {connect} from 'react-redux'
+import { connect } from "react-redux";
 import {
   Divider,
   Message,
@@ -14,31 +14,31 @@ import {
   TextArea,
   Checkbox,
   Header,
-} from 'semantic-ui-react'
+} from "semantic-ui-react";
-import {useHistory, useParams, Link} from 'react-router-dom'
+import { useHistory, useParams, Link } from "react-router-dom";
-import {concat, of, from} from 'rxjs'
+import { concat, of, from } from "rxjs";
-import {pluck, distinctUntilChanged, map, switchMap} from 'rxjs/operators'
+import { pluck, distinctUntilChanged, map, switchMap } from "rxjs/operators";
-import {useObservable} from 'rxjs-hooks'
+import { useObservable } from "rxjs-hooks";
-import {findInput} from 'utils'
+import { findInput } from "utils";
-import {useForm, Controller} from 'react-hook-form'
+import { useForm, Controller } from "react-hook-form";
-import {useTranslation, Trans as Translate} from 'react-i18next'
+import { useTranslation, Trans as Translate } from "react-i18next";
-import Markdown from 'react-markdown'
+import Markdown from "react-markdown";
 
-import api from 'api'
+import api from "api";
-import {Page, FileUploadField} from 'components'
+import { Page, FileUploadField } from "components";
-import type {Track} from 'types'
+import type { Track } from "types";
 
-import {FileUploadStatus} from 'pages/UploadPage'
+import { FileUploadStatus } from "pages/UploadPage";
 
 function ReplaceTrackData({ slug }) {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
-  const [file, setFile] = React.useState(null)
+  const [file, setFile] = React.useState(null);
-  const [result, setResult] = React.useState(null)
+  const [result, setResult] = React.useState(null);
-  const onComplete = React.useCallback((_id, r) => setResult(r), [setResult])
+  const onComplete = React.useCallback((_id, r) => setResult(r), [setResult]);
 
   return (
     <>
-      <Header as="h2">{t('TrackEditor.replaceTrackData')}</Header>
+      <Header as="h2">{t("TrackEditor.replaceTrackData")}</Header>
       {!file ? (
         <FileUploadField onSelect={setFile} />
       ) : result ? (
@@ -51,73 +51,74 @@ function ReplaceTrackData({slug}) {
         <FileUploadStatus {...{ file, onComplete, slug }} />
       )}
     </>
-  )
+  );
 }
 
-const TrackEditor = connect((state) => ({login: state.login}))(function TrackEditor({login}) {
+const TrackEditor = connect((state) => ({ login: state.login }))(
-  const {t} = useTranslation()
+  function TrackEditor({ login }) {
-  const [busy, setBusy] = React.useState(false)
+    const { t } = useTranslation();
-  const {register, control, handleSubmit} = useForm()
+    const [busy, setBusy] = React.useState(false);
-  const {slug} = useParams()
+    const { register, control, handleSubmit } = useForm();
-  const history = useHistory()
+    const { slug } = useParams();
+    const history = useHistory();
 
     const track: null | Track = useObservable(
       (_$, args$) => {
-      const slug$ = args$.pipe(pluck(0), distinctUntilChanged())
+        const slug$ = args$.pipe(pluck(0), distinctUntilChanged());
         return slug$.pipe(
           map((slug) => `/tracks/${slug}`),
           switchMap((url) => concat(of(null), from(api.get(url)))),
-        pluck('track')
+          pluck("track")
-      )
+        );
       },
       null,
       [slug]
-  )
+    );
 
-  const loading = busy || track == null
+    const loading = busy || track == null;
-  const isAuthor = login?.id === track?.author?.id
+    const isAuthor = login?.id === track?.author?.id;
 
     // Navigate to track detials if we are not the author
     React.useEffect(() => {
       if (!login || (track && !isAuthor)) {
-      history.replace(`/tracks/${slug}`)
+        history.replace(`/tracks/${slug}`);
       }
-  }, [slug, login, track, isAuthor, history])
+    }, [slug, login, track, isAuthor, history]);
 
     const onSubmit = React.useMemo(
       () =>
         handleSubmit(async (values) => {
-        setBusy(true)
+          setBusy(true);
 
           try {
             await api.put(`/tracks/${slug}`, {
               body: {
-              track: _.pickBy(values, (v) => typeof v !== 'undefined'),
+                track: _.pickBy(values, (v) => typeof v !== "undefined"),
               },
-          })
+            });
-          history.push(`/tracks/${slug}`)
+            history.push(`/tracks/${slug}`);
           } finally {
-          setBusy(false)
+            setBusy(false);
           }
         }),
       [slug, handleSubmit, history]
-  )
+    );
 
-  const [confirmDelete, setConfirmDelete] = React.useState(false)
+    const [confirmDelete, setConfirmDelete] = React.useState(false);
     const onDelete = React.useCallback(async () => {
-    setBusy(true)
+      setBusy(true);
 
       try {
-      await api.delete(`/tracks/${slug}`)
+        await api.delete(`/tracks/${slug}`);
-      history.push('/tracks')
+        history.push("/tracks");
       } finally {
-      setConfirmDelete(false)
+        setConfirmDelete(false);
-      setBusy(false)
+        setBusy(false);
       }
-  }, [setBusy, setConfirmDelete, slug, history])
+    }, [setBusy, setConfirmDelete, slug, history]);
 
-  const trackTitle: string = track?.title || t('general.unnamedTrack')
+    const trackTitle: string = track?.title || t("general.unnamedTrack");
-  const title = t('TrackEditor.title', {trackTitle})
+    const title = t("TrackEditor.title", { trackTitle });
 
     return (
       <Page title={title}>
@@ -127,23 +128,42 @@ const TrackEditor = connect((state) => ({login: state.login}))(function TrackEdi
               <Header as="h2">{title}</Header>
               <Form loading={loading} key={track?.slug} onSubmit={onSubmit}>
                 <Ref innerRef={findInput(register)}>
-                <Form.Input label="Title" name="title" defaultValue={track?.title} style={{fontSize: '120%'}} />
+                  <Form.Input
+                    label="Title"
+                    name="title"
+                    defaultValue={track?.title}
+                    style={{ fontSize: "120%" }}
+                  />
                 </Ref>
 
                 <Form.Field>
-                <label>{t('TrackEditor.description.label')}</label>
+                  <label>{t("TrackEditor.description.label")}</label>
                   <Ref innerRef={register}>
-                  <TextArea name="description" rows={4} defaultValue={track?.description} />
+                    <TextArea
+                      name="description"
+                      rows={4}
+                      defaultValue={track?.description}
+                    />
                   </Ref>
                 </Form.Field>
 
                 <Form.Field>
                   <label>
-                  {t('TrackEditor.visibility.label')}
+                    {t("TrackEditor.visibility.label")}
                     <Popup
                       wide="very"
-                    content={<Markdown>{t('TrackEditor.visibility.description')}</Markdown>}
+                      content={
-                    trigger={<Icon name="warning sign" style={{marginLeft: 8}} color="orange" />}
+                        <Markdown>
+                          {t("TrackEditor.visibility.description")}
+                        </Markdown>
+                      }
+                      trigger={
+                        <Icon
+                          name="warning sign"
+                          style={{ marginLeft: 8 }}
+                          color="orange"
+                        />
+                      }
                     />
                   </label>
 
@@ -154,14 +174,14 @@ const TrackEditor = connect((state) => ({login: state.login}))(function TrackEdi
                     render={(props) => (
                       <Checkbox
                         name="public"
-                      label={t('TrackEditor.visibility.checkboxLabel')}
+                        label={t("TrackEditor.visibility.checkboxLabel")}
                         checked={props.value}
                         onChange={(_, { checked }) => props.onChange(checked)}
                       />
                     )}
                   />
                 </Form.Field>
-              <Button type="submit">{t('general.save')}</Button>
+                <Button type="submit">{t("general.save")}</Button>
               </Form>
             </Grid.Column>
             <Grid.Column width={6}>
@@ -169,25 +189,26 @@ const TrackEditor = connect((state) => ({login: state.login}))(function TrackEdi
 
               <Divider />
 
-            <Header as="h2">{t('TrackEditor.dangerZone.title')}</Header>
+              <Header as="h2">{t("TrackEditor.dangerZone.title")}</Header>
-            <Markdown>{t('TrackEditor.dangerZone.description')}</Markdown>
+              <Markdown>{t("TrackEditor.dangerZone.description")}</Markdown>
 
               <Button color="red" onClick={() => setConfirmDelete(true)}>
-              {t('general.delete')}
+                {t("general.delete")}
               </Button>
               <Confirm
                 open={confirmDelete}
                 onCancel={() => setConfirmDelete(false)}
                 onConfirm={onDelete}
-              content={t('TrackEditor.dangerZone.confirmDelete')}
+                content={t("TrackEditor.dangerZone.confirmDelete")}
-              confirmButton={t('general.delete')}
+                confirmButton={t("general.delete")}
-              cancelButton={t('general.cancel')}
+                cancelButton={t("general.cancel")}
               />
             </Grid.Column>
           </Grid.Row>
         </Grid>
       </Page>
-  )
+    );
-})
+  }
+);
 
-export default TrackEditor
+export default TrackEditor;

frontend/src/pages/TrackPage/TrackActions.tsx

@@ -1,10 +1,10 @@
 import React from 'react'
 import {Link} from 'react-router-dom'
 import {Icon, Popup, Button, Dropdown} from 'semantic-ui-react'
-import {useTranslation} from 'react-i18next'
+import { useTranslation } from "react-i18next";
 
 export default function TrackActions({slug, isAuthor, onDownload}) {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
 
   return (
     <>
@@ -25,11 +25,7 @@ export default function TrackActions({slug, isAuthor, onDownload}) {
 
       <Dropdown text={t('TrackPage.actions.download')} button>
         <Dropdown.Menu>
-          <Dropdown.Item
+          <Dropdown.Item text={t('TrackPage.actions.original')}onClick={() => onDownload('original.csv')} disabled={!isAuthor} />
-            text={t('TrackPage.actions.original')}
-            onClick={() => onDownload('original.csv')}
-            disabled={!isAuthor}
-          />
           <Dropdown.Item text={t('TrackPage.actions.gpx')} onClick={() => onDownload('track.gpx')} />
         </Dropdown.Menu>
       </Dropdown>

frontend/src/pages/TrackPage/TrackComments.tsx

@@ -1,34 +1,57 @@
-import React from 'react'
+import React from "react";
-import {Message, Segment, Form, Button, Loader, Header, Comment} from 'semantic-ui-react'
+import {
-import Markdown from 'react-markdown'
+  Message,
-import {useTranslation} from 'react-i18next'
+  Segment,
+  Form,
+  Button,
+  Loader,
+  Header,
+  Comment,
+} from "semantic-ui-react";
+import Markdown from "react-markdown";
+import { useTranslation } from "react-i18next";
 
-import {Avatar, FormattedDate} from 'components'
+import { Avatar, FormattedDate } from "components";
 
 function CommentForm({ onSubmit }) {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
-  const [body, setBody] = React.useState('')
+  const [body, setBody] = React.useState("");
 
   const onSubmitComment = React.useCallback(() => {
-    onSubmit({body})
+    onSubmit({ body });
-    setBody('')
+    setBody("");
-  }, [onSubmit, body])
+  }, [onSubmit, body]);
 
   return (
     <Form reply onSubmit={onSubmitComment}>
-      <Form.TextArea rows={4} value={body} onChange={(e) => setBody(e.target.value)} />
+      <Form.TextArea
-      <Button content={t('TrackPage.comments.post')} labelPosition="left" icon="edit" primary />
+        rows={4}
+        value={body}
+        onChange={(e) => setBody(e.target.value)}
+      />
+      <Button
+        content={t("TrackPage.comments.post")}
+        labelPosition="left"
+        icon="edit"
+        primary
+      />
     </Form>
-  )
+  );
 }
 
-export default function TrackComments({comments, onSubmit, onDelete, login, hideLoader}) {
+export default function TrackComments({
-  const {t} = useTranslation()
+  comments,
+  onSubmit,
+  onDelete,
+  login,
+  hideLoader,
+}) {
+  const { t } = useTranslation();
   return (
     <>
       <Comment.Group>
         <Header as="h2" dividing>
-          {t('TrackPage.comments.title')}
+          {t("TrackPage.comments.title")}
         </Header>
 
         <Loader active={!hideLoader && comments == null} inline />
@@ -37,7 +60,9 @@ export default function TrackComments({comments, onSubmit, onDelete, login, hide
           <Comment key={comment.id}>
             <Avatar user={comment.author} />
             <Comment.Content>
-              <Comment.Author as="a">{comment.author.displayName}</Comment.Author>
+              <Comment.Author as="a">
+                {comment.author.displayName}
+              </Comment.Author>
               <Comment.Metadata>
                 <div>
                   <FormattedDate date={comment.createdAt} relative />
@@ -50,11 +75,11 @@ export default function TrackComments({comments, onSubmit, onDelete, login, hide
                 <Comment.Actions>
                   <Comment.Action
                     onClick={(e) => {
-                      onDelete(comment.id)
+                      onDelete(comment.id);
-                      e.preventDefault()
+                      e.preventDefault();
                     }}
                   >
-                    {t('general.delete')}
+                    {t("general.delete")}
                   </Comment.Action>
                 </Comment.Actions>
               )}
@@ -62,10 +87,12 @@ export default function TrackComments({comments, onSubmit, onDelete, login, hide
           </Comment>
         ))}
 
-        {comments != null && !comments.length && <Message>{t('TrackPage.comments.empty')}</Message>}
+        {comments != null && !comments.length && (
+          <Message>{t("TrackPage.comments.empty")}</Message>
+        )}
 
         {login && comments != null && <CommentForm onSubmit={onSubmit} />}
       </Comment.Group>
     </>
-  )
+  );
 }

frontend/src/pages/TrackPage/TrackDetails.tsx

@@ -1,54 +1,85 @@
-import React from 'react'
+import React from "react";
-import _ from 'lodash'
+import _ from "lodash";
-import {List, Header, Grid} from 'semantic-ui-react'
+import { List, Header, Grid } from "semantic-ui-react";
-import {Duration} from 'luxon'
+import { Duration } from "luxon";
-import {useTranslation} from 'react-i18next'
+import { useTranslation } from "react-i18next";
 
-import {FormattedDate, Visibility} from 'components'
+import { FormattedDate, Visibility } from "components";
-import {formatDistance, formatDuration} from 'utils'
+
+function formatDuration(seconds) {
+  return Duration.fromMillis((seconds ?? 0) * 1000).toFormat("h'h' mm'm'");
+}
 
 export default function TrackDetails({ track, isAuthor }) {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
 
   const items = [
-    track.public != null && isAuthor && [t('TrackPage.details.visibility'), <Visibility public={track.public} />],
+    track.public != null &&
+      isAuthor && [
+        t("TrackPage.details.visibility"),
+        <Visibility public={track.public} />,
+      ],
 
-    track.uploadedByUserAgent != null && [t('TrackPage.details.uploadedWith'), track.uploadedByUserAgent],
+    track.uploadedByUserAgent != null && [
+      t("TrackPage.details.uploadedWith"),
+      track.uploadedByUserAgent,
+    ],
 
-    track.duration != null && [t('TrackPage.details.duration'), formatDuration(track.duration)],
+    track.duration != null && [
+      t("TrackPage.details.duration"),
+      formatDuration(track.duration),
+    ],
 
-    track.createdAt != null && [t('TrackPage.details.uploadedDate'), <FormattedDate date={track.createdAt} />],
+    track.createdAt != null && [
+      t("TrackPage.details.uploadedDate"),
+      <FormattedDate date={track.createdAt} />,
+    ],
 
-    track?.recordedAt != null && [t('TrackPage.details.recordedDate'), <FormattedDate date={track?.recordedAt} />],
+    track?.recordedAt != null && [
+      t("TrackPage.details.recordedDate"),
+      <FormattedDate date={track?.recordedAt} />,
+    ],
 
-    track?.numEvents != null && [t('TrackPage.details.numEvents'), track?.numEvents],
+    track?.numEvents != null && [
+      t("TrackPage.details.numEvents"),
+      track?.numEvents,
+    ],
 
-    track?.length != null && [t('TrackPage.details.length'), formatDistance(track?.length)],
+    track?.length != null && [
+      t("TrackPage.details.length"),
+      `${(track?.length / 1000).toFixed(2)} km`,
+    ],
 
     track?.processingStatus != null &&
-      track?.processingStatus != 'error' && [t('TrackPage.details.processingStatus'), track.processingStatus],
+      track?.processingStatus != "error" && [
+        t("TrackPage.details.processingStatus"),
+        track.processingStatus,
+      ],
 
-    track.originalFileName != null && [t('TrackPage.details.originalFileName'), <code>{track.originalFileName}</code>],
+    track.originalFileName != null && [
-  ].filter(Boolean)
+      t("TrackPage.details.originalFileName"),
+      <code>{track.originalFileName}</code>,
+    ],
+  ].filter(Boolean);
 
-  const COLUMNS = 4
+  const COLUMNS = 4;
   const chunkSize = Math.ceil(items.length / COLUMNS)
   return (
     <Grid>
       <Grid.Row columns={COLUMNS}>
       {_.chunk(items, chunkSize).map((chunkItems, idx) => (
             <Grid.Column key={idx}>
+
   <List>
     {chunkItems.map(([title, value]) => (
       <List.Item key={title}>
       <List.Header>{title}</List.Header>
       <List.Description>{value}</List.Description>
-                </List.Item>
+      </List.Item>))}
-              ))}
     </List>
             </Grid.Column>
           ))}
         </Grid.Row>
     </Grid>
-  )
+  );
 }

frontend/src/pages/TrackPage/TrackMap.tsx

@@ -61,6 +61,7 @@ export default function TrackMap({
                   layout: {
                     'text-field': ['number-format', ['get', p], {'min-fraction-digits': 2, 'max-fraction-digits': 2}],
                     'text-allow-overlap': true,
+                    'text-font': ['Open Sans Bold', 'Arial Unicode MS Regular'],
                     'text-size': 14,
                     'text-keep-upright': false,
                     'text-anchor': a,

frontend/src/pages/TrackPage/index.tsx

@@ -1,5 +1,5 @@
-import React from 'react'
+import React from "react";
-import {connect} from 'react-redux'
+import { connect } from "react-redux";
 import {
   List,
   Dropdown,
@@ -12,128 +12,146 @@ import {
   Message,
   Confirm,
   Container,
-} from 'semantic-ui-react'
+} from "semantic-ui-react";
-import {useParams, useHistory} from 'react-router-dom'
+import { useParams, useHistory } from "react-router-dom";
-import {concat, combineLatest, of, from, Subject} from 'rxjs'
+import { concat, combineLatest, of, from, Subject } from "rxjs";
-import {pluck, distinctUntilChanged, map, switchMap, startWith, catchError} from 'rxjs/operators'
+import {
-import {useObservable} from 'rxjs-hooks'
+  pluck,
-import Markdown from 'react-markdown'
+  distinctUntilChanged,
-import {useTranslation} from 'react-i18next'
+  map,
+  switchMap,
+  startWith,
+  catchError,
+} from "rxjs/operators";
+import { useObservable } from "rxjs-hooks";
+import Markdown from "react-markdown";
+import { useTranslation } from "react-i18next";
 
-import api from 'api'
+import api from "api";
-import {Page} from 'components'
+import { Page } from "components";
-import type {Track, TrackData, TrackComment} from 'types'
+import type { Track, TrackData, TrackComment } from "types";
-import {trackLayer, trackLayerRaw} from '../../mapstyles'
+import { trackLayer, trackLayerRaw } from "../../mapstyles";
 
-import TrackActions from './TrackActions'
+import TrackActions from "./TrackActions";
-import TrackComments from './TrackComments'
+import TrackComments from "./TrackComments";
-import TrackDetails from './TrackDetails'
+import TrackDetails from "./TrackDetails";
-import TrackMap from './TrackMap'
+import TrackMap from "./TrackMap";
 
-import styles from './TrackPage.module.less'
+import styles from "./TrackPage.module.less";
 
 function useTriggerSubject() {
-  const subject$ = React.useMemo(() => new Subject(), [])
+  const subject$ = React.useMemo(() => new Subject(), []);
-  const trigger = React.useCallback(() => subject$.next(null), [subject$])
+  const trigger = React.useCallback(() => subject$.next(null), [subject$]);
-  return [trigger, subject$]
+  return [trigger, subject$];
 }
 
-function TrackMapSettings({showTrack, setShowTrack, pointsMode, setPointsMode, side, setSide}) {
+function TrackMapSettings({
-  const {t} = useTranslation()
+  showTrack,
+  setShowTrack,
+  pointsMode,
+  setPointsMode,
+  side,
+  setSide,
+}) {
+  const { t } = useTranslation();
   return (
     <>
-      <Header as="h4">{t('TrackPage.mapSettings.title')}</Header>
+      <Header as="h4">{t("TrackPage.mapSettings.title")}</Header>
       <List>
         <List.Item>
-          <Checkbox checked={showTrack} onChange={(e, d) => setShowTrack(d.checked)} />{' '}
+          <Checkbox
-          {t('TrackPage.mapSettings.showTrack')}
+            checked={showTrack}
+            onChange={(e, d) => setShowTrack(d.checked)}
+          />{" "}
+          {t("TrackPage.mapSettings.showTrack")}
           <div style={{ marginTop: 8 }}>
             <span
               style={{
-                borderTop: '3px dashed ' + trackLayerRaw.paint['line-color'],
+                borderTop: "3px dashed " + trackLayerRaw.paint["line-color"],
                 height: 0,
                 width: 24,
-                display: 'inline-block',
+                display: "inline-block",
-                verticalAlign: 'middle',
+                verticalAlign: "middle",
                 marginRight: 4,
               }}
             />
-            {t('TrackPage.mapSettings.gpsTrack')}
+            {t("TrackPage.mapSettings.gpsTrack")}
           </div>
           <div>
             <span
               style={{
-                borderTop: '6px solid ' + trackLayerRaw.paint['line-color'],
+                borderTop: "6px solid " + trackLayerRaw.paint["line-color"],
                 height: 6,
                 width: 24,
-                display: 'inline-block',
+                display: "inline-block",
-                verticalAlign: 'middle',
+                verticalAlign: "middle",
                 marginRight: 4,
               }}
             />
-            {t('TrackPage.mapSettings.snappedTrack')}
+            {t("TrackPage.mapSettings.snappedTrack")}
           </div>
         </List.Item>
         <List.Item>
-          <List.Header> {t('TrackPage.mapSettings.points')} </List.Header>
+          <List.Header> {t("TrackPage.mapSettings.points")} </List.Header>
           <Dropdown
             selection
             value={pointsMode}
             onChange={(e, d) => setPointsMode(d.value)}
             options={[
-              {key: 'none', value: 'none', text: 'None'},
+              { key: "none", value: "none", text: "None" },
               {
-                key: 'overtakingEvents',
+                key: "overtakingEvents",
-                value: 'overtakingEvents',
+                value: "overtakingEvents",
-                text: t('TrackPage.mapSettings.confirmedPoints'),
+                text: t("TrackPage.mapSettings.confirmedPoints"),
               },
               {
-                key: 'measurements',
+                key: "measurements",
-                value: 'measurements',
+                value: "measurements",
-                text: t('TrackPage.mapSettings.allPoints'),
+                text: t("TrackPage.mapSettings.allPoints"),
               },
             ]}
           />
         </List.Item>
         <List.Item>
-          <List.Header>{t('TrackPage.mapSettings.side')}</List.Header>
+          <List.Header>{t("TrackPage.mapSettings.side")}</List.Header>
           <Dropdown
             selection
             value={side}
             onChange={(e, d) => setSide(d.value)}
             options={[
               {
-                key: 'overtaker',
+                key: "overtaker",
-                value: 'overtaker',
+                value: "overtaker",
-                text: t('TrackPage.mapSettings.overtakerSide'),
+                text: t("TrackPage.mapSettings.overtakerSide"),
               },
               {
-                key: 'stationary',
+                key: "stationary",
-                value: 'stationary',
+                value: "stationary",
-                text: t('TrackPage.mapSettings.stationarySide'),
+                text: t("TrackPage.mapSettings.stationarySide"),
               },
             ]}
           />
         </List.Item>
       </List>
     </>
-  )
+  );
 }
 
-const TrackPage = connect((state) => ({login: state.login}))(function TrackPage({login}) {
+const TrackPage = connect((state) => ({ login: state.login }))(
-  const {slug} = useParams()
+  function TrackPage({ login }) {
-  const {t} = useTranslation()
+    const { slug } = useParams();
+    const { t } = useTranslation();
 
-  const [reloadComments, reloadComments$] = useTriggerSubject()
+    const [reloadComments, reloadComments$] = useTriggerSubject();
-  const history = useHistory()
+    const history = useHistory();
 
     const data: {
-    track: null | Track
+      track: null | Track;
-    trackData: null | TrackData
+      trackData: null | TrackData;
-    comments: null | TrackComment[]
+      comments: null | TrackComment[];
     } | null = useObservable(
       (_$, args$) => {
-      const slug$ = args$.pipe(pluck(0), distinctUntilChanged())
+        const slug$ = args$.pipe(pluck(0), distinctUntilChanged());
         const track$ = slug$.pipe(
           map((slug) => `/tracks/${slug}`),
           switchMap((url) =>
@@ -141,13 +159,13 @@ const TrackPage = connect((state) => ({login: state.login}))(function TrackPage(
               of(null),
               from(api.get(url)).pipe(
                 catchError(() => {
-                history.replace('/tracks')
+                  history.replace("/tracks");
                 })
               )
             )
           ),
-        pluck('track')
+          pluck("track")
-      )
+        );
 
         const trackData$ = slug$.pipe(
           map((slug) => `/tracks/${slug}/data`),
@@ -156,13 +174,13 @@ const TrackPage = connect((state) => ({login: state.login}))(function TrackPage(
               of(undefined),
               from(api.get(url)).pipe(
                 catchError(() => {
-                return of(null)
+                  return of(null);
                 })
               )
             )
           ),
           startWith(undefined) // show track infos before track data is loaded
-      )
+        );
 
         const comments$ = concat(of(null), reloadComments$).pipe(
           switchMap(() => slug$),
@@ -170,13 +188,13 @@ const TrackPage = connect((state) => ({login: state.login}))(function TrackPage(
           switchMap((url) =>
             from(api.get(url)).pipe(
               catchError(() => {
-              return of(null)
+                return of(null);
               })
             )
           ),
-        pluck('comments'),
+          pluck("comments"),
           startWith(null) // show track infos before comments are loaded
-      )
+        );
 
         return combineLatest([track$, trackData$, comments$]).pipe(
           map(([track, trackData, comments]) => ({
@@ -184,60 +202,65 @@ const TrackPage = connect((state) => ({login: state.login}))(function TrackPage(
             trackData,
             comments,
           }))
-      )
+        );
       },
       null,
       [slug]
-  )
+    );
 
     const onSubmitComment = React.useCallback(
       async ({ body }) => {
         await api.post(`/tracks/${slug}/comments`, {
           body: { comment: { body } },
-      })
+        });
-      reloadComments()
+        reloadComments();
       },
       [slug, reloadComments]
-  )
+    );
 
     const onDeleteComment = React.useCallback(
       async (id) => {
-      await api.delete(`/tracks/${slug}/comments/${id}`)
+        await api.delete(`/tracks/${slug}/comments/${id}`);
-      reloadComments()
+        reloadComments();
       },
       [slug, reloadComments]
-  )
+    );
 
-  const [downloadError, setDownloadError] = React.useState(null)
+    const [downloadError, setDownloadError] = React.useState(null);
-  const hideDownloadError = React.useCallback(() => setDownloadError(null), [setDownloadError])
+    const hideDownloadError = React.useCallback(
+      () => setDownloadError(null),
+      [setDownloadError]
+    );
     const onDownload = React.useCallback(
       async (filename) => {
         try {
-        await api.downloadFile(`/tracks/${slug}/download/${filename}`)
+          await api.downloadFile(`/tracks/${slug}/download/${filename}`);
         } catch (err) {
           if (/Failed to fetch/.test(String(err))) {
-          setDownloadError(t('TrackPage.downloadError'))
+            setDownloadError(t("TrackPage.downloadError"));
           } else {
-          setDownloadError(String(err))
+            setDownloadError(String(err));
           }
         }
       },
       [slug]
-  )
+    );
 
-  const isAuthor = login?.id === data?.track?.author?.id
+    const isAuthor = login?.id === data?.track?.author?.id;
 
-  const {track, trackData, comments} = data || {}
+    const { track, trackData, comments } = data || {};
 
-  const loading = track == null || trackData === undefined
+    const loading = track == null || trackData === undefined;
-  const processing = ['processing', 'queued', 'created'].includes(track?.processingStatus)
+    const processing = ["processing", "queued", "created"].includes(
-  const error = track?.processingStatus === 'error'
+      track?.processingStatus
+    );
+    const error = track?.processingStatus === "error";
 
-  const [showTrack, setShowTrack] = React.useState(true)
+    const [showTrack, setShowTrack] = React.useState(true);
-  const [pointsMode, setPointsMode] = React.useState('overtakingEvents') // none|overtakingEvents|measurements
+    const [pointsMode, setPointsMode] = React.useState("overtakingEvents"); // none|overtakingEvents|measurements
-  const [side, setSide] = React.useState('overtaker') // overtaker|stationary
+    const [side, setSide] = React.useState("overtaker"); // overtaker|stationary
 
-  const title = track ? track.title || t('general.unnamedTrack') : null
+    const title = track ? track.title || t("general.unnamedTrack") : null;
     return (
       <Page
         title={title}
@@ -248,14 +271,14 @@ const TrackPage = connect((state) => ({login: state.login}))(function TrackPage(
                 <Segment basic>
                   <div
                     style={{
-                    display: 'flex',
+                      display: "flex",
-                    alignItems: 'baseline',
+                      alignItems: "baseline",
                       marginBlockStart: 32,
                       marginBlockEnd: 16,
                     }}
                   >
                     <Header as="h1">{title}</Header>
-                  <div style={{marginLeft: 'auto'}}>
+                    <div style={{ marginLeft: "auto" }}>
                       <TrackActions {...{ isAuthor, onDownload, slug }} />
                     </div>
                   </div>
@@ -269,7 +292,10 @@ const TrackPage = connect((state) => ({login: state.login}))(function TrackPage(
             <div className={styles.stage}>
               <Loader active={loading} />
               <Dimmer.Dimmable blurring dimmed={loading}>
-              <TrackMap {...{track, trackData, pointsMode, side, showTrack}} style={{height: '80vh'}} />
+                <TrackMap
+                  {...{ track, trackData, pointsMode, side, showTrack }}
+                  style={{ height: "80vh" }}
+                />
               </Dimmer.Dimmable>
 
               <div className={styles.details}>
@@ -288,13 +314,17 @@ const TrackPage = connect((state) => ({login: state.login}))(function TrackPage(
 
                 {processing && (
                   <Message warning>
-                  <Message.Content>{t('TrackPage.processing')}</Message.Content>
+                    <Message.Content>
+                      {t("TrackPage.processing")}
+                    </Message.Content>
                   </Message>
                 )}
 
                 {error && (
                   <Message error>
-                  <Message.Content>{t('TrackPage.processingError')}</Message.Content>
+                    <Message.Content>
+                      {t("TrackPage.processingError")}
+                    </Message.Content>
                   </Message>
                 )}
               </div>
@@ -304,7 +334,7 @@ const TrackPage = connect((state) => ({login: state.login}))(function TrackPage(
               {track?.description && (
                 <>
                   <Header as="h2" dividing>
-                  {t('TrackPage.description')}
+                    {t("TrackPage.description")}
                   </Header>
                   <Markdown>{track.description}</Markdown>
                 </>
@@ -323,12 +353,13 @@ const TrackPage = connect((state) => ({login: state.login}))(function TrackPage(
           open={downloadError != null}
           cancelButton={false}
           onConfirm={hideDownloadError}
-        header={t('TrackPage.downloadFailed')}
+          header={t("TrackPage.downloadFailed")}
           content={String(downloadError)}
-        confirmButton={t('general.ok')}
+          confirmButton={t("general.ok")}
         />
       </Page>
-  )
+    );
-})
+  }
+);
 
-export default TrackPage
+export default TrackPage;

frontend/src/pages/TracksPage.tsx

@@ -1,49 +1,65 @@
-import React, {useCallback} from 'react'
+import React, { useCallback } from "react";
-import {connect} from 'react-redux'
+import { connect } from "react-redux";
-import {Button, Message, Item, Header, Loader, Pagination, Icon} from 'semantic-ui-react'
+import {
-import {useObservable} from 'rxjs-hooks'
+  Button,
-import {Link} from 'react-router-dom'
+  Message,
-import {of, from, concat} from 'rxjs'
+  Item,
-import {map, switchMap, distinctUntilChanged} from 'rxjs/operators'
+  Header,
-import _ from 'lodash'
+  Loader,
-import {useTranslation, Trans as Translate} from 'react-i18next'
+  Pagination,
+  Icon,
+} from "semantic-ui-react";
+import { useObservable } from "rxjs-hooks";
+import { Link } from "react-router-dom";
+import { of, from, concat } from "rxjs";
+import { map, switchMap, distinctUntilChanged } from "rxjs/operators";
+import _ from "lodash";
+import { useTranslation, Trans as Translate } from "react-i18next";
 
-import type {Track} from 'types'
+import type { Track } from "types";
-import {Avatar, Page, StripMarkdown, FormattedDate, Visibility} from 'components'
+import {
-import api from 'api'
+  Avatar,
-import {useQueryParam} from 'query'
+  Page,
+  StripMarkdown,
+  FormattedDate,
+  Visibility,
+} from "components";
+import api from "api";
+import { useQueryParam } from "query";
 
 function TrackList({ privateTracks }: { privateTracks: boolean }) {
-  const [page, setPage] = useQueryParam<number>('page', 1, Number)
+  const [page, setPage] = useQueryParam<number>("page", 1, Number);
 
-  const pageSize = 10
+  const pageSize = 10;
 
   const data: {
-    tracks: Track[]
+    tracks: Track[];
-    trackCount: number
+    trackCount: number;
   } | null = useObservable(
     (_$, inputs$) =>
       inputs$.pipe(
         map(([page, privateTracks]) => {
-          const url = '/tracks' + (privateTracks ? '/feed' : '')
+          const url = "/tracks" + (privateTracks ? "/feed" : "");
-          const query = {limit: pageSize, offset: pageSize * (page - 1)}
+          const query = { limit: pageSize, offset: pageSize * (page - 1) };
-          return {url, query}
+          return { url, query };
         }),
         distinctUntilChanged(_.isEqual),
-        switchMap((request) => concat(of(null), from(api.get(request.url, {query: request.query}))))
+        switchMap((request) =>
+          concat(of(null), from(api.get(request.url, { query: request.query })))
+        )
       ),
     null,
     [page, privateTracks]
-  )
+  );
 
-  const {tracks, trackCount} = data || {tracks: [], trackCount: 0}
+  const { tracks, trackCount } = data || { tracks: [], trackCount: 0 };
-  const loading = !data
+  const loading = !data;
-  const totalPages = Math.ceil(trackCount / pageSize)
+  const totalPages = Math.ceil(trackCount / pageSize);
-  const {t} = useTranslation()
+  const { t } = useTranslation();
 
   return (
     <div>
-      <Loader content={t('general.loading')} active={loading} />
+      <Loader content={t("general.loading")} active={loading} />
       {!loading && totalPages > 1 && (
         <Pagination
           activePage={page}
@@ -62,7 +78,7 @@ function TrackList({privateTracks}: {privateTracks: boolean}) {
         <NoPublicTracksMessage />
       )}
     </div>
-  )
+  );
 }
 
 export function NoPublicTracksMessage() {
@@ -72,27 +88,27 @@ export function NoPublicTracksMessage() {
         No public tracks yet. <Link to="/upload">Upload the first!</Link>
       </Translate>
     </Message>
-  )
+  );
 }
 
 function maxLength(t: string | null, max: number): string | null {
   if (t && t.length > max) {
-    return t.substring(0, max) + ' ...'
+    return t.substring(0, max) + " ...";
   } else {
-    return t
+    return t;
   }
 }
 
 const COLOR_BY_STATUS = {
-  error: 'red',
+  error: "red",
-  complete: 'green',
+  complete: "green",
-  created: 'gray',
+  created: "gray",
-  queued: 'orange',
+  queued: "orange",
-  processing: 'orange',
+  processing: "orange",
-}
+};
 
 export function TrackListItem({ track, privateTracks = false }) {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
 
   return (
     <Item key={track.slug}>
@@ -101,10 +117,14 @@ export function TrackListItem({track, privateTracks = false}) {
       </Item.Image>
       <Item.Content>
         <Item.Header as={Link} to={`/tracks/${track.slug}`}>
-          {track.title || t('general.unnamedTrack')}
+          {track.title || t("general.unnamedTrack")}
         </Item.Header>
         <Item.Meta>
-          {privateTracks ? null : <span>{t('TracksPage.createdBy', {author: track.author.displayName})}</span>}
+          {privateTracks ? null : (
+            <span>
+              {t("TracksPage.createdBy", { author: track.author.displayName })}
+            </span>
+          )}
           <span>
             <FormattedDate date={track.createdAt} />
           </span>
@@ -116,36 +136,48 @@ export function TrackListItem({track, privateTracks = false}) {
           <Item.Extra>
             <Visibility public={track.public} />
 
-            <span style={{marginLeft: '1em'}}>
+            <span style={{ marginLeft: "1em" }}>
-              <Icon color={COLOR_BY_STATUS[track.processingStatus]} name="bolt" fitted />{' '}
+              <Icon
+                color={COLOR_BY_STATUS[track.processingStatus]}
+                name="bolt"
+                fitted
+              />{" "}
               {t(`TracksPage.processing.${track.processingStatus}`)}
             </span>
           </Item.Extra>
         )}
       </Item.Content>
     </Item>
-  )
+  );
 }
 
 function UploadButton({ navigate, ...props }) {
-  const {t} = useTranslation()
+  const { t } = useTranslation();
   const onClick = useCallback(
     (e) => {
-      e.preventDefault()
+      e.preventDefault();
-      navigate()
+      navigate();
     },
     [navigate]
-  )
+  );
   return (
-    <Button onClick={onClick} {...props} color="green" style={{float: 'right'}}>
+    <Button
-      {t('TracksPage.upload')}
+      onClick={onClick}
+      {...props}
+      color="green"
+      style={{ float: "right" }}
+    >
+      {t("TracksPage.upload")}
     </Button>
-  )
+  );
 }
 
-const TracksPage = connect((state) => ({login: (state as any).login}))(function TracksPage({login, privateTracks}) {
+const TracksPage = connect((state) => ({ login: (state as any).login }))(
-  const {t} = useTranslation()
+  function TracksPage({ login, privateTracks }) {
-  const title = privateTracks ? t('TracksPage.titleUser') : t('TracksPage.titlePublic')
+    const { t } = useTranslation();
+    const title = privateTracks
+      ? t("TracksPage.titleUser")
+      : t("TracksPage.titlePublic");
 
     return (
       <Page title={title}>
@@ -153,7 +185,8 @@ const TracksPage = connect((state) => ({login: (state as any).login}))(function
         {privateTracks && <Link component={UploadButton} to="/upload" />}
         <TrackList {...{ privateTracks }} />
       </Page>
-  )
+    );
-})
+  }
+);
 
-export default TracksPage
+export default TracksPage;

frontend/src/pages/UploadPage.tsx

@@ -1,46 +1,46 @@
-import _ from 'lodash'
+import _ from "lodash";
-import React from 'react'
+import React from "react";
-import {Header, List, Loader, Table, Icon} from 'semantic-ui-react'
+import { List, Loader, Table, Icon } from "semantic-ui-react";
-import {Link} from 'react-router-dom'
+import { Link } from "react-router-dom";
-import {useTranslation} from 'react-i18next'
+import { useTranslation } from "react-i18next";
 
-import {FileUploadField, Page} from 'components'
+import { FileUploadField, Page } from "components";
-import type {Track} from 'types'
+import type { Track } from "types";
-import api from 'api'
+import api from "api";
-import configPromise from 'config'
+import configPromise from "config";
 
 function isSameFile(a: File, b: File) {
-  return a.name === b.name && a.size === b.size
+  return a.name === b.name && a.size === b.size;
 }
 
 function formatFileSize(bytes: number) {
   if (bytes < 1024) {
-    return `${bytes} bytes`
+    return `${bytes} bytes`;
   }
 
-  bytes /= 1024
+  bytes /= 1024;
 
   if (bytes < 1024) {
-    return `${bytes.toFixed(1)} KiB`
+    return `${bytes.toFixed(1)} KiB`;
   }
 
-  bytes /= 1024
+  bytes /= 1024;
 
   if (bytes < 1024) {
-    return `${bytes.toFixed(1)} MiB`
+    return `${bytes.toFixed(1)} MiB`;
   }
 
-  bytes /= 1024
+  bytes /= 1024;
-  return `${bytes.toFixed(1)} GiB`
+  return `${bytes.toFixed(1)} GiB`;
 }
 
 type FileUploadResult =
   | {
-      track: Track
+      track: Track;
     }
   | {
-      errors: Record<string, string>
+      errors: Record<string, string>;
-    }
+    };
 
 export function FileUploadStatus({
   id,
@@ -48,111 +48,121 @@ export function FileUploadStatus({
   onComplete,
   slug,
 }: {
-  id: string
+  id: string;
-  file: File
+  file: File;
-  onComplete: (id: string, result: FileUploadResult) => void
+  onComplete: (id: string, result: FileUploadResult) => void;
-  slug?: string
+  slug?: string;
 }) {
-  const [progress, setProgress] = React.useState(0)
+  const [progress, setProgress] = React.useState(0);
 
   React.useEffect(
     () => {
-      let xhr
+      let xhr;
 
       async function _work() {
-        const formData = new FormData()
+        const formData = new FormData();
-        formData.append('body', file)
+        formData.append("body", file);
 
-        xhr = new XMLHttpRequest()
+        xhr = new XMLHttpRequest();
-        xhr.withCredentials = true
+        xhr.withCredentials = true;
 
         const onProgress = (e) => {
-          const progress = (e.loaded || 0) / (e.total || 1)
+          const progress = (e.loaded || 0) / (e.total || 1);
-          setProgress(progress)
+          setProgress(progress);
-        }
+        };
 
         const onLoad = (e) => {
-          onComplete(id, xhr.response)
+          onComplete(id, xhr.response);
-        }
+        };
 
-        xhr.responseType = 'json'
+        xhr.responseType = "json";
-        xhr.onload = onLoad
+        xhr.onload = onLoad;
-        xhr.upload.onprogress = onProgress
+        xhr.upload.onprogress = onProgress;
 
-        const config = await configPromise
+        const config = await configPromise;
         if (slug) {
-          xhr.open('PUT', `${config.apiUrl}/tracks/${slug}`)
+          xhr.open("PUT", `${config.apiUrl}/tracks/${slug}`);
         } else {
-          xhr.open('POST', `${config.apiUrl}/tracks`)
+          xhr.open("POST", `${config.apiUrl}/tracks`);
         }
 
         // const accessToken = await api.getValidAccessToken()
 
         // xhr.setRequestHeader('Authorization', accessToken)
-        xhr.send(formData)
+        xhr.send(formData);
       }
 
-      _work()
+      _work();
-      return () => xhr.abort()
+      return () => xhr.abort();
     },
     // eslint-disable-next-line react-hooks/exhaustive-deps
     [file]
-  )
+  );
 
-  const {t} = useTranslation()
+  const { t } = useTranslation();
   return (
     <span>
-      <Loader inline size="mini" active />{' '}
+      <Loader inline size="mini" active />{" "}
       {progress < 1
-        ? t('UploadPage.uploadProgress', {
+        ? t("UploadPage.uploadProgress", {
             progress: (progress * 100).toFixed(0),
           })
-        : t('UploadPage.processing')}
+        : t("UploadPage.processing")}
     </span>
-  )
+  );
 }
 
 type FileEntry = {
-  id: string
+  id: string;
-  file?: File | null
+  file?: File | null;
-  size: number
+  size: number;
-  name: string
+  name: string;
-  result?: FileUploadResult
+  result?: FileUploadResult;
-}
+};
 
 export default function UploadPage() {
-  const [files, setFiles] = React.useState<FileEntry[]>([])
+  const [files, setFiles] = React.useState<FileEntry[]>([]);
 
   const onCompleteFileUpload = React.useCallback(
     (id, result) => {
-      setFiles((files) => files.map((file) => (file.id === id ? {...file, result, file: null} : file)))
+      setFiles((files) =>
+        files.map((file) =>
+          file.id === id ? { ...file, result, file: null } : file
+        )
+      );
     },
     [setFiles]
-  )
+  );
 
   function onSelectFiles(fileList) {
     const newFiles = Array.from(fileList).map((file) => ({
-      id: 'file-' + String(Math.floor(Math.random() * 1000000)),
+      id: "file-" + String(Math.floor(Math.random() * 1000000)),
       file,
       name: file.name,
       size: file.size,
-    }))
+    }));
-    setFiles(files.filter((a) => !newFiles.some((b) => isSameFile(a, b))).concat(newFiles))
+    setFiles(
+      files
+        .filter((a) => !newFiles.some((b) => isSameFile(a, b)))
+        .concat(newFiles)
+    );
   }
 
-  const {t} = useTranslation()
+  const { t } = useTranslation();
 
-  const title = t('UploadPage.title')
   return (
-    <Page title={title}>
+    <Page title="Upload">
-      <Header as="h1">{title}</Header>
       {files.length ? (
         <Table>
           <Table.Header>
             <Table.Row>
-              <Table.HeaderCell>{t('UploadPage.table.filename')}</Table.HeaderCell>
+              <Table.HeaderCell>
-              <Table.HeaderCell>{t('UploadPage.table.size')}</Table.HeaderCell>
+                {t("UploadPage.table.filename")}
-              <Table.HeaderCell>{t('UploadPage.table.statusTitle')}</Table.HeaderCell>
+              </Table.HeaderCell>
+              <Table.HeaderCell>{t("UploadPage.table.size")}</Table.HeaderCell>
+              <Table.HeaderCell>
+                {t("UploadPage.table.statusTitle")}
+              </Table.HeaderCell>
               <Table.HeaderCell colSpan={2}></Table.HeaderCell>
             </Table.Row>
           </Table.Header>
@@ -169,7 +179,9 @@ export default function UploadPage() {
                   {result?.errors ? (
                     <List>
                       {_.sortBy(Object.entries(result.errors))
-                        .filter(([field, message]) => typeof message === 'string')
+                        .filter(
+                          ([field, message]) => typeof message === "string"
+                        )
                         .map(([field, message]) => (
                           <List.Item key={field}>
                             <List.Icon name="warning sign" color="red" />
@@ -179,17 +191,29 @@ export default function UploadPage() {
                     </List>
                   ) : result ? (
                     <>
-                      <Icon name="check" /> {result.track?.title || t('general.unnamedTrack')}
+                      <Icon name="check" />{" "}
+                      {result.track?.title || t("general.unnamedTrack")}
                     </>
                   ) : (
-                    <FileUploadStatus {...{id, file}} onComplete={onCompleteFileUpload} />
+                    <FileUploadStatus
+                      {...{ id, file }}
+                      onComplete={onCompleteFileUpload}
+                    />
                   )}
                 </Table.Cell>
                 <Table.Cell>
-                  {result?.track ? <Link to={`/tracks/${result.track.slug}`}>{t('general.show')}</Link> : null}
+                  {result?.track ? (
+                    <Link to={`/tracks/${result.track.slug}`}>
+                      {t("general.show")}
+                    </Link>
+                  ) : null}
                 </Table.Cell>
                 <Table.Cell>
-                  {result?.track ? <Link to={`/tracks/${result.track.slug}/edit`}>{t('general.edit')}</Link> : null}
+                  {result?.track ? (
+                    <Link to={`/tracks/${result.track.slug}/edit`}>
+                      {t("general.edit")}
+                    </Link>
+                  ) : null}
                 </Table.Cell>
               </Table.Row>
             ))}
@@ -199,5 +223,5 @@ export default function UploadPage() {
 
       <FileUploadField onSelect={onSelectFiles} multiple />
     </Page>
-  )
+  );
 }

frontend/src/pages/index.js

@@ -1,4 +1,3 @@
-export {default as AcknowledgementsPage} from './AcknowledgementsPage'
 export {default as ExportPage} from './ExportPage'
 export {default as HomePage} from './HomePage'
 export {default as LoginRedirectPage} from './LoginRedirectPage'
@@ -9,5 +8,4 @@ export {default as SettingsPage} from './SettingsPage'
 export {default as TrackEditor} from './TrackEditor'
 export {default as TrackPage} from './TrackPage'
 export {default as TracksPage} from './TracksPage'
-export {default as MyTracksPage} from './MyTracksPage'
 export {default as UploadPage} from './UploadPage'

frontend/src/query.ts

@@ -53,7 +53,7 @@ export function useQueryParam<T extends QueryValue>(
 ): [T, (newValue: T) => void] {
   const history = useHistory()
   useLocation() // to trigger a reload when the url changes
-  const {[name]: value = defaultValue} = parseQuery(history.location.search) as unknown as {
+  const {[name]: value = defaultValue} = (parseQuery(history.location.search) as unknown) as {
     [name: string]: T
   }
   const setter = useMemo(

frontend/src/reducers/mapConfig.ts

@@ -1,92 +1,108 @@
-import {useMemo} from 'react'
+import { useMemo } from "react";
-import {useSelector} from 'react-redux'
+import { useSelector } from "react-redux";
-import produce from 'immer'
+import produce from "immer";
-import _ from 'lodash'
+import _ from "lodash";
 
-type BaseMapStyle = 'positron' | 'bright'
+type BaseMapStyle = "positron" | "bright";
 
 type RoadAttribute =
-  | 'distance_overtaker_mean'
+  | "distance_overtaker_mean"
-  | 'distance_overtaker_min'
+  | "distance_overtaker_min"
-  | 'distance_overtaker_max'
+  | "distance_overtaker_max"
-  | 'distance_overtaker_median'
+  | "distance_overtaker_median"
-  | 'overtaking_event_count'
+  | "overtaking_event_count"
-  | 'usage_count'
+  | "usage_count"
-  | 'zone'
+  | "zone";
 
 export type MapConfig = {
   baseMap: {
-    style: BaseMapStyle
+    style: BaseMapStyle;
-  }
+  };
   obsRoads: {
-    show: boolean
+    show: boolean;
-    showUntagged: boolean
+    showUntagged: boolean;
-    attribute: RoadAttribute
+    attribute: RoadAttribute;
-    maxCount: number
+    maxCount: number;
-  }
+  };
   obsEvents: {
-    show: boolean
+    show: boolean;
-  }
+  };
-  obsRegions: {
-    show: boolean
-  }
   filters: {
-    currentUser: boolean
+    currentUser: boolean;
-    dateMode: 'none' | 'range' | 'threshold'
+    dateMode: "none" | "range" | "threshold";
-    startDate?: null | string
+    startDate?: null | string;
-    endDate?: null | string
+    endDate?: null | string;
-    thresholdAfter?: null | boolean
+    thresholdAfter?: null | boolean;
-  }
+  };
-}
+  obsRegions: {
+    show: boolean;
+  };
+  filters: {
+    currentUser: boolean;
+    dateMode: "none" | "range" | "threshold";
+    startDate?: null | string;
+    endDate?: null | string;
+    thresholdAfter?: null | boolean;
+  };
+};
 
 export const initialState: MapConfig = {
   baseMap: {
-    style: 'positron',
+    style: "positron",
   },
   obsRoads: {
     show: true,
     showUntagged: true,
-    attribute: 'distance_overtaker_median',
+    attribute: "distance_overtaker_median",
     maxCount: 20,
   },
   obsEvents: {
     show: false,
   },
-  obsRegions: {
-    show: true,
-  },
   filters: {
     currentUser: false,
-    dateMode: 'none',
+    dateMode: "none",
     startDate: null,
     endDate: null,
     thresholdAfter: true,
   },
-}
+  obsRegions: {
+    show: true,
+  },
+};
 
 type MapConfigAction = {
-  type: 'MAP_CONFIG.SET_FLAG'
+  type: "MAP_CONFIG.SET_FLAG";
-  payload: {flag: string; value: any}
+  payload: { flag: string; value: any };
-}
+};
 
-export function setMapConfigFlag(flag: string, value: unknown): MapConfigAction {
+export function setMapConfigFlag(
-  return {type: 'MAP_CONFIG.SET_FLAG', payload: {flag, value}}
+  flag: string,
+  value: unknown
+): MapConfigAction {
+  return { type: "MAP_CONFIG.SET_FLAG", payload: { flag, value } };
 }
 
 export function useMapConfig() {
-  const mapConfig = useSelector((state) => state.mapConfig)
+  const mapConfig = useSelector((state) => state.mapConfig);
-  const result = useMemo(() => _.merge({}, initialState, mapConfig), [mapConfig])
+  const result = useMemo(
-  return result
+    () => _.merge({}, initialState, mapConfig),
+    [mapConfig]
+  );
+  return result;
 }
 
-export default function mapConfigReducer(state: MapConfig = initialState, action: MapConfigAction) {
+export default function mapConfigReducer(
+  state: MapConfig = initialState,
+  action: MapConfigAction
+) {
   switch (action.type) {
-    case 'MAP_CONFIG.SET_FLAG':
+    case "MAP_CONFIG.SET_FLAG":
       return produce(state, (draft) => {
-        _.set(draft, action.payload.flag, action.payload.value)
+        _.set(draft, action.payload.flag, action.payload.value);
-      })
+      });
 
     default:
-      return state
+      return state;
   }
 }

frontend/src/semantic-ui/site/elements/header.overrides

@@ -3,19 +3,22 @@
 *******************************/
 
 h1.ui.header {
+  font-family: "Open Sans Condensed";
   line-height: 35pt;
   font-size: 30pt;
   color: @obsColorB4;
 }
 
 h2.ui.header {
-  font-weight: 400;
+  font-family: "Open Sans";
+  font-weight: 300;
   line-height: 25pt;
   font-size: 20pt;
   color: @obsColorG1;
 }
 
 h3.ui.header {
+  font-family: "Open Sans";
   font-weight: normal;
   line-height: 18pt;
   font-size: 15pt;
@@ -23,6 +26,7 @@ h3.ui.header {
 }
 
 h4.ui.header {
+  font-family: "Open Sans";
   font-weight: bold;
   line-height: 15pt;
   font-size: 15pt;

frontend/src/semantic-ui/site/elements/list.overrides

@@ -1,12 +1,3 @@
 /*******************************
     User Variable Overrides
 *******************************/
-
-
-.ui.list{
-  .item{
-      .header{
-        margin-bottom: 5px;
-      }
-  }
-}

frontend/src/semantic-ui/site/globals/site.variables

@@ -3,8 +3,8 @@
 *******************************/
 
 @importGoogleFonts : false;
-@fontName          : 'Open Sans', 'Noto Sans', 'Roboto', -apple-system, BlinkMacSystemFont, 'Segoe UI', 'Oxygen', 'Ubuntu', 'Cantarell',
+@fontName          : 'Open Sans';
-                     'Fira Sans', 'Droid Sans', 'Helvetica Neue', sans-serif;
+
 @obsColorB4: #114594;
 @obsColorG1: #76520E;
 @obsColorW: #FFFFFF;

frontend/src/semantic-ui/site/views/statistic.overrides

@@ -1,7 +1,3 @@
 /*******************************
     User Variable Overrides
 *******************************/
-
-.ui.statistic .label{
-  font-size: 13px;
-}

frontend/src/styles.less

@@ -9,11 +9,7 @@
 @secondaryColor: @obsColorG1;
 @borderColor: #e0e0e0;
 
-@fontFamilyDefault: 'Open Sans', 'Noto Sans', 'Roboto', -apple-system, BlinkMacSystemFont, 'Segoe UI', 'Oxygen', 'Ubuntu', 'Cantarell',
-'Fira Sans', 'Droid Sans', 'Helvetica Neue', sans-serif;
-
 @menuHeight: 50px;
-@menuHeightMobile: 220px;
 
 @mobile: ~'screen and (max-width: 767px)';
 @desktop: ~'screen and (min-width: 768px)';