Skip to content
This repository was archived by the owner on Aug 21, 2025. It is now read-only.

Open TermsOfService and PrivacyPolicy in a customTab#88

Merged
hamorillo merged 1 commit intotrunkfrom
hamorillo/GRA-630
Jul 29, 2025
Merged

Open TermsOfService and PrivacyPolicy in a customTab#88
hamorillo merged 1 commit intotrunkfrom
hamorillo/GRA-630

Conversation

@hamorillo
Copy link

Description

TermOfService and PrivacyPolicy links should be opened in an in-app browser so the user doesn't leave the app context.

Kapture.2025-07-29.at.12.54.28.mp4

Testing Steps

  • Launch de app
  • Open the About this app dialog
  • Verify TermOfService and PrivacyPolicy are opened in an in-app browser.

@hamorillo hamorillo requested review from Copilot and etoledom July 29, 2025 10:55
@hamorillo hamorillo added the enhancement New feature or request label Jul 29, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements in-app browser functionality for Terms of Service and Privacy Policy links to keep users within the app context when viewing these documents.

  • Adds CustomTabs support using the AndroidX browser library
  • Updates Terms of Service and Privacy Policy link handlers to use in-app browser instead of external browser
  • Introduces a new openUrlInApp function that uses CustomTabsIntent for in-app browsing

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
AboutAppDialog.kt Updates link handlers to use CustomTabs and adds new in-app URL opening function
build.gradle.kts Adds AndroidX browser dependency for CustomTabs functionality

@hamorillo hamorillo force-pushed the hamorillo/GRA-630 branch from 0c59ac2 to 9559c60 Compare July 29, 2025 10:56
Copy link
Contributor

@etoledom etoledom left a comment

Choose a reason for hiding this comment

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

Working perfectly as described 🎉

Those links shouldn't leave the app, so use CustomTabs to open them inside the app.
@hamorillo hamorillo force-pushed the hamorillo/GRA-630 branch from 9559c60 to 4b2ac13 Compare July 29, 2025 20:09
@hamorillo hamorillo merged commit 7b16255 into trunk Jul 29, 2025
10 checks passed
@hamorillo hamorillo deleted the hamorillo/GRA-630 branch July 29, 2025 20:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants