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(landing): Add Config Debug for screenshot elevation data. #3174

Merged
merged 7 commits into from
Mar 13, 2024

Conversation

Wentao-Kuang
Copy link
Contributor

@Wentao-Kuang Wentao-Kuang commented Mar 8, 2024

Motivation

As we want to screenshot some elevation map for smoke tests. We need enable the debug urls for it.

Modification

  • Add debug.layer for screenshot different output.
  • Add dropdown for debug.layer
  • Add debug.terrain for screenshot map with terrain on.
  • Add dropdown for debug.terrain
  • Add bearing and pitch in the location url hash
  • Comment out the updateVisibleLayers, date range slider is not been used atm. This is cause problematic by automatically adding aerial layers.

Checklist

If not applicable, provide explanation of why.

  • Tests updated
  • Docs updated
  • Issue linked in Title

@Wentao-Kuang Wentao-Kuang requested a review from a team as a code owner March 8, 2024 00:53
@Wentao-Kuang Wentao-Kuang requested a review from blacha March 8, 2024 00:53
packages/geo/src/__tests__/slug.test.ts Outdated Show resolved Hide resolved
packages/geo/src/__tests__/slug.test.ts Outdated Show resolved Hide resolved
packages/landing/src/components/debug.tsx Outdated Show resolved Hide resolved
packages/landing/src/config.map.ts Show resolved Hide resolved
packages/landing/src/config.debug.ts Show resolved Hide resolved
packages/landing/src/components/debug.tsx Outdated Show resolved Hide resolved
@Wentao-Kuang Wentao-Kuang added this pull request to the merge queue Mar 13, 2024
Merged via the queue into master with commit 0ee360d Mar 13, 2024
10 checks passed
@Wentao-Kuang Wentao-Kuang deleted the feat/elevation-screenshot branch March 13, 2024 02:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants