Skip to content

Update GitHub link and improve URL handling in about dialog#6

Merged
Inkesk-Dozing merged 1 commit intoupdate-about-sectionfrom
copilot/sub-pr-4
Feb 24, 2026
Merged

Update GitHub link and improve URL handling in about dialog#6
Inkesk-Dozing merged 1 commit intoupdate-about-sectionfrom
copilot/sub-pr-4

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 23, 2026

The about dialog linked to an invalid GitHub URL (with .git suffix), and ThemeService.openUrl lacked error handling. Android was also missing the required intent permissions to launch URLs.

Changes

  • Fix GitHub URL: Remove .git extension from the repository link in the about dialog
  • Android permissions: Add QUERY_ALL_PACKAGES / URL launcher intent queries to AndroidManifest.xml so url_launcher can resolve HTTP/HTTPS intents on Android 11+
  • Error handling: Wrap ThemeService.openUrl with try/catch to gracefully handle launch failures
  • .gitignore: Add Flutter-specific entries to avoid committing build artifacts and IDE files

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Update GitHub link and improve error handling Update GitHub link and improve URL handling in about dialog Feb 23, 2026
Copilot AI requested a review from Inkesk-Dozing February 23, 2026 19:29
@Inkesk-Dozing Inkesk-Dozing marked this pull request as ready for review February 24, 2026 01:45
@Inkesk-Dozing Inkesk-Dozing merged commit bccee77 into update-about-section Feb 24, 2026
1 check passed
@Inkesk-Dozing Inkesk-Dozing deleted the copilot/sub-pr-4 branch February 24, 2026 15:50
Inkesk-Dozing added a commit that referenced this pull request Mar 2, 2026
Squashed 5 commits from this day:
- Merge pull request #6 from eOzkull/copilot/sub-pr-4
- Merge branch 'main' into update-about-section
- Merge pull request #4 from eOzkull/update-about-section
- Merge branch 'main' into StillnWater-patch-1
- Merge pull request #3 from eOzkull/StillnWater-patch-1
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.

2 participants