Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(viewer): make zoom level configurable in viewer for OAPIF #221

Merged
merged 4 commits into from
Aug 8, 2024

Conversation

rkettelerij
Copy link
Collaborator

@rkettelerij rkettelerij commented Aug 6, 2024

Description

Introduce the web settings:

ogcApi:
  features:
    collections:
      - id: foo
        metadata:
          title: Foo bar
          description: Foo bar baz
        web:
          featuresViewer:
            minScale: 3000
            maxScale: 40000
          featureViewer:
            minScale: 1000
            maxScale: 40000

Type of change

  • New feature

Checklist:

  • I've double-checked the code in this PR myself
  • I've left the code better than before (boy scout rule)
  • The code is readable, comments are added that explain hard or non-obvious parts.
  • I've expanded/improved the (unit) tests, when applicable
  • I've run (unit) tests that prove my solution works
  • There's no sensitive information like credentials in my PR

@rkettelerij rkettelerij force-pushed the configurable-viewer-zoom branch from 6085c59 to f7b0759 Compare August 8, 2024 10:59
@rkettelerij rkettelerij marked this pull request as ready for review August 8, 2024 11:00
@rkettelerij rkettelerij force-pushed the configurable-viewer-zoom branch from 2466d79 to 1a944da Compare August 8, 2024 11:28
config/config.go Show resolved Hide resolved
@rkettelerij rkettelerij merged commit 0835244 into master Aug 8, 2024
5 checks passed
@rkettelerij rkettelerij deleted the configurable-viewer-zoom branch August 8, 2024 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants