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

Update offline maps section for Collect v2024.2 #1821

Merged
merged 6 commits into from
Jul 8, 2024

Conversation

lognaturel
Copy link
Member

This page is linked from Collect so it's a common entry point into docs. For this reason, I invested some time to try to make it useful to a broad range of audiences.

I also tried to describe special cases we've considered and make recommendations on how to use the functionality based on some forum conversations.

@alyblenkin and @grzesiek2010 you may want to give this a quick read to see whether I forgot anything.

docs/collect-offline-maps.rst Outdated Show resolved Hide resolved
docs/collect-offline-maps.rst Outdated Show resolved Hide resolved
docs/collect-offline-maps.rst Outdated Show resolved Hide resolved
docs/collect-offline-maps.rst Outdated Show resolved Hide resolved
docs/collect-offline-maps.rst Outdated Show resolved Hide resolved
docs/collect-offline-maps.rst Outdated Show resolved Hide resolved
docs/collect-offline-maps.rst Outdated Show resolved Hide resolved
docs/collect-offline-maps.rst Outdated Show resolved Hide resolved
docs/collect-offline-maps.rst Outdated Show resolved Hide resolved
docs/collect-offline-maps.rst Outdated Show resolved Hide resolved
@lognaturel lognaturel requested a review from yanokwa July 5, 2024 22:09
@alyblenkin
Copy link

Looks good! I really like the framing at the top around why this is helpful and the overall structure.

A few thoughts that came to mind (might be beyond the scope)

  • Guiding people on how to change the file name because some might be quite similar
  • The note around deletion is helpful. I wonder if people will want to know how you determine the file size to see if you should free up space
  • Nice to have: it might be helpful to see a gif at the top of the entire flow? Or an image with the different screens to show the steps all together.

@lognaturel
Copy link
Member Author

Thanks, @alyblenkin! 1 and 2 are going to depend on how the MBTiles files are being generated, Android version, etc so I think they're out of scope. I like the idea of experimenting with a gif showing the whole flow! I'll merge this and file a separate issue.

@lognaturel lognaturel merged commit 9381398 into getodk:master Jul 8, 2024
1 check passed
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.

3 participants