Skip to content

prompt user before allowing location access#9

Merged
jpcloureiro merged 2 commits into
mainfrom
location-permission-dialog
Mar 26, 2024
Merged

prompt user before allowing location access#9
jpcloureiro merged 2 commits into
mainfrom
location-permission-dialog

Conversation

@jpcloureiro

Copy link
Copy Markdown

No description provided.

@jpcloureiro

Copy link
Copy Markdown
Author

depends on #8.

ready to review after rebase

@tommasini tommasini left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@sethkfman sethkfman left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sethkfman sethkfman left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jpcloureiro

Copy link
Copy Markdown
Author

QA passed, wait on #8 to merge

@jpcloureiro jpcloureiro merged commit 64cb789 into main Mar 26, 2024
@jpcloureiro jpcloureiro deleted the location-permission-dialog branch March 26, 2024 16:41
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.

3 participants