Skip to content

Conversation

Jack251970
Copy link
Member

From #3982.

Use !File.Exists(Path.Combine(PortableDataPath, DeletionIndicatorFile))) instead of !File.Exists(DeletionIndicatorFile)) so that we can check if deletion indicator file exists.

@prlabeler prlabeler bot added the bug Something isn't working label Sep 21, 2025
@Jack251970 Jack251970 requested a review from Copilot September 21, 2025 12:08
@github-actions github-actions bot added this to the 2.1.0 milestone Sep 21, 2025
@Jack251970 Jack251970 requested a review from jjw24 September 21, 2025 12:08
Copy link
Contributor

@Copilot 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

Fixes a bug in the portable mode detection logic where the deletion indicator file check was using an incorrect path. The fix ensures that the deletion indicator file is properly located within the portable data directory.

  • Corrects the path for checking the deletion indicator file existence
  • Adds readonly modifier to static fields for better immutability
  • Improves code readability by splitting the condition check across multiple lines

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
Flow.Launcher/SettingPages/ViewModels/SettingsPaneGeneralViewModel.cs Adds readonly modifier to the _portableMode field
Flow.Launcher.Infrastructure/UserSettings/DataLocation.cs Fixes deletion indicator file path check and adds readonly modifiers to path fields

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Jack251970 Jack251970 modified the milestones: 2.1.0, 2.0.1 Sep 21, 2025
Copy link

gitstream-cm bot commented Sep 21, 2025

🥷 Code experts: jjw24, onesounds

Jack251970, onesounds have most 👩‍💻 activity in the files.
Jack251970, jjw24 have most 🧠 knowledge in the files.

See details

Flow.Launcher.Infrastructure/UserSettings/DataLocation.cs

Activity based on git-commit:

Jack251970 onesounds
SEP
AUG
JUL
JUN
MAY
APR

Knowledge based on git-blame:
jjw24: 70%
Jack251970: 20%

Flow.Launcher/SettingPages/ViewModels/SettingsPaneGeneralViewModel.cs

Activity based on git-commit:

Jack251970 onesounds
SEP 3 additions & 1 deletions
AUG
JUL 8 additions & 8 deletions
JUN 2 additions & 0 deletions
MAY 15 additions & 4 deletions
APR 64 additions & 137 deletions 355 additions & 225 deletions

Knowledge based on git-blame:
Jack251970: 25%

✨ Comment /gs review for LinearB AI review. Learn how to automate it here.

Copy link

gitstream-cm bot commented Sep 21, 2025

Be a legend 🏆 by adding a before and after screenshot of the changes you made, especially if they are around UI/UX.

@Jack251970 Jack251970 merged commit 35867a0 into dev Sep 21, 2025
12 checks passed
@Jack251970 Jack251970 deleted the portable_mode_fix branch September 21, 2025 12:23
@jjw24 jjw24 modified the milestones: 2.0.1, 2.1.0 Sep 21, 2025
TBM13 pushed a commit to TBM13/Flow.Launcher that referenced this pull request Sep 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants