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

Removed Stamen maps #5822

Merged
merged 4 commits into from
Nov 16, 2023
Merged

Removed Stamen maps #5822

merged 4 commits into from
Nov 16, 2023

Conversation

grzesiek2010
Copy link
Member

@grzesiek2010 grzesiek2010 commented Nov 6, 2023

Closes #5806

Why is this the best possible solution? Were any other approaches considered?

This is what we agreed on. We don't think Stamen maps are very popular among our users so we can get rid of them.

How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?

I think we need to test:

  • that Stamen maps are not available now
  • upgrading from a version with Stamen maps selected in settings (this should be migrated to OSM)
  • scanning/importing QR codes with Stamen maps on an apk where they are removed (this pr)

Do we need any specific form for testing your changes? If so, please attach one.

No.

Does this change require updates to documentation? If so, please file an issue here and include the link below.

No.

Before submitting this PR, please make sure you have:

  • added or modified tests for any new or changed behavior
  • run ./gradlew checkAll and confirmed all checks still pass OR confirm CircleCI build passes and run ./gradlew connectedDebugAndroidTest locally.
  • added a comment above any new strings describing it for translators
  • verified that any code or assets from external sources are properly credited in comments and/or in the about file.
  • verified that any new UI elements use theme colors. UI Components Style guidelines

@grzesiek2010 grzesiek2010 marked this pull request as ready for review November 6, 2023 14:32
@grzesiek2010 grzesiek2010 requested a review from seadowg November 6, 2023 14:32
@grzesiek2010 grzesiek2010 requested a review from seadowg November 9, 2023 08:50
@dbemke
Copy link

dbemke commented Nov 14, 2023

What is the expected in this case? I created (via reconfigure QR code) a QR code in 2023.3.0 of the demo project with Stamen maps. I installed Collect with the PR version (build in AS- with maps available) and added a new project by scanning the created QR code. The result in the PR version is that Google maps are source of maps.
So what is the default map instead of Stamen maps when there is an app with all maps are available (apk built in Android Studio) and an app without all token for maps (CircleCi apk- this changes Stamen to Google and Google to OSM as Google in not available?)?

@grzesiek2010
Copy link
Member Author

What is the expected in this case? I created (via reconfigure QR code) a QR code in 2023.3.0 of the demo project with Stamen maps. I installed Collect with the PR version (build in AS- with maps available) and added a new project by scanning the created QR code. The result in the PR version is that Google maps are source of maps.
So what is the default map instead of Stamen maps when there is an app with all maps are available (apk built in Android Studio) and an app without all token for maps (CircleCi apk- this changes Stamen to Google and Google to OSM as Google in not available?)?

as long as Google Maps are available I would say it's not a big deal although it was supposed to use OSM... could you give it a try now?

@dbemke
Copy link

dbemke commented Nov 16, 2023

Tested with Success!

Verified on a device with Android 10

Verified cases:

@srujner
Copy link

srujner commented Nov 16, 2023

Tested with Success!

Verified on a device with Android 13

@grzesiek2010 grzesiek2010 merged commit c5fd96e into getodk:master Nov 16, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

As of July 31, 2023, Stamen Map Tiles are served by Stadia Maps
4 participants