Skip to content

feat: Add extension to open customtab browser#759

Merged
baillyjamy merged 3 commits intomainfrom
feat-customtab-browser
May 6, 2026
Merged

feat: Add extension to open customtab browser#759
baillyjamy merged 3 commits intomainfrom
feat-customtab-browser

Conversation

@baillyjamy
Copy link
Copy Markdown
Contributor

Add extension to open customtab browser

Copy link
Copy Markdown

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 introduces a new Context extension to open URLs in an Android Custom Tab and updates the Privacy Management screen to use it for the “Source code” link.

Changes:

  • Added Context.openUrlInCustomTab(url) in Common (using androidx.browser Custom Tabs).
  • Updated PrivacyManagementHomeContent to open the source URL via Custom Tabs instead of a generic browser intent.
  • Added androidx.browser:browser to the version catalog and Common module dependencies.

Reviewed changes

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

File Description
PrivacyManagement/src/main/kotlin/com/infomaniak/core/privacymanagement/screencontent/PrivacyManagementHomeContent.kt Switches the source button click action to use the new Custom Tab URL opener.
gradle/core.versions.toml Adds the androidx.browser library and its version to the catalog.
Common/src/main/kotlin/com/infomaniak/core/common/extensions/ContextExt.kt Adds openUrlInCustomTab extension (currently without the same safety/fallback behavior as openUrl).
Common/build.gradle.kts Adds the androidx.browser dependency to support Custom Tabs usage in Common.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 6, 2026

@baillyjamy baillyjamy merged commit ad73bfd into main May 6, 2026
7 checks passed
@baillyjamy baillyjamy deleted the feat-customtab-browser branch May 6, 2026 12:48
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