[PR #7791] [CLOSED] chore: add detailed logs in the deeplink handler #8117

New issue

Closed

opened 2026-03-23 23:22:41 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:22:41 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/7791
Author: @LucasXu0
Created: 4/21/2025
Status: ❌ Closed

Base: main ← Head: deep_link_logs

📝 Commits (1)

e9e8792 chore: add detailed logs in the deeplink handler

📊 Changes

2 files changed (+42 additions, -11 deletions)

View changed files

📝 frontend/appflowy_flutter/lib/core/helpers/url_launcher.dart (+23 -9)
📝 frontend/appflowy_flutter/lib/user/application/auth/af_cloud_auth_service.dart (+19 -2)

📄 Description

Feature Preview

PR Checklist

My code adheres to AppFlowy's Conventions
I've listed at least one issue that this PR fixes in the description above.
I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes.
All existing tests are passing.

Summary by Sourcery

Add detailed logging to the deeplink handler to improve error tracking and debugging

Bug Fixes:

Add null checking for OAuth URL to prevent potential crashes

Enhancements:

Improve error handling in URL launching and deep link processing by adding more comprehensive logging

Chores:

Add detailed log messages to capture potential failure points in authentication and URL launching processes

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/AppFlowy-IO/AppFlowy/pull/7791 **Author:** [@LucasXu0](https://github.com/LucasXu0) **Created:** 4/21/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `deep_link_logs` --- ### 📝 Commits (1) - [`e9e8792`](https://github.com/AppFlowy-IO/AppFlowy/commit/e9e8792302568164542340c389d71ce5db0489b9) chore: add detailed logs in the deeplink handler ### 📊 Changes **2 files changed** (+42 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/core/helpers/url_launcher.dart` (+23 -9) 📝 `frontend/appflowy_flutter/lib/user/application/auth/af_cloud_auth_service.dart` (+19 -2) </details> ### 📄 Description   ### Feature Preview  ---  #### PR Checklist - [ ] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [ ] I've listed at least one issue that this PR fixes in the description above. - [ ] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes. - [ ] All existing tests are passing. ## Summary by Sourcery Add detailed logging to the deeplink handler to improve error tracking and debugging Bug Fixes: - Add null checking for OAuth URL to prevent potential crashes Enhancements: - Improve error handling in URL launching and deep link processing by adding more comprehensive logging Chores: - Add detailed log messages to capture potential failure points in authentication and URL launching processes --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>