Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug: Quickask - Jan processes dangling in background after quitting on Windows #2356

Closed
freelerobot opened this issue Mar 14, 2024 · 4 comments · Fixed by #2642
Closed

bug: Quickask - Jan processes dangling in background after quitting on Windows #2356

freelerobot opened this issue Mar 14, 2024 · 4 comments · Fixed by #2642
Assignees
Labels
P1: important Important feature / fix type: bug Something isn't working
Milestone

Comments

@freelerobot
Copy link
Contributor

freelerobot commented Mar 14, 2024

Describe the bug
A clear and concise description of what the bug is.

-v0.4.8-322

  • windows amd ryzen
  • 4070s

Steps to reproduce
Steps to reproduce the behavior:

  1. Open App
  2. Click X
  3. 5 Jan processes remain running, as indicated in task manager
image
  1. type cmd+j
  2. The quick ask loader animation shows for a few seconds
  3. Then nothing happens. The app doesn't open, quick ask doesn't finish loading.
image
  1. Open the app from bottom nav again
  2. Try Quick Ask
  3. Broken Entirely

Why it's a problem

Jan disappears from TaskManager > Apps, so the app is theoretically not running.
But the 5 dangling processes are forever running as background processes.
Albeit not taking up much resources

Questions

  • Can we name processes more distinctly? So we can tell from names which one is running
  • What is the current expected behavior when users click x on windows?
@freelerobot freelerobot added P1: important Important feature / fix type: bug Something isn't working platform: Windows labels Mar 14, 2024
@freelerobot freelerobot changed the title bug: bad app lifecycle on Windows bug: dangling process after quitting on Windows Mar 14, 2024
@freelerobot freelerobot changed the title bug: dangling process after quitting on Windows bug: all processes dangling after quitting on Windows Mar 14, 2024
@Van-QA Van-QA added this to the v0.4.10 milestone Mar 14, 2024
@Van-QA Van-QA changed the title bug: all processes dangling after quitting on Windows bug: Quickask - Jan processes dangling in background after quitting on Windows Mar 19, 2024
@louis-menlo louis-menlo moved this to Planned in Menlo Mar 22, 2024
@louis-menlo louis-menlo moved this from Planned to In Progress in Menlo Mar 25, 2024
@Van-QA Van-QA modified the milestones: v0.4.10, v0.4.11 Mar 27, 2024
@namchuai
Copy link
Collaborator

namchuai commented Apr 1, 2024

Currently, if we are enabling QuickAsk, we will display Jan in tray. Which means Jan still running even user clicked close on main window.

I think multiple processes shown in task manager is normal for electron app. Visual studio code has same behavior.
Image

But the issue that you captured (QuickAsk display installing extension) is a bug. We troubleshoot this issue in PR #2543

I will retest, follow your steps and update.

@freelerobot
Copy link
Contributor Author

freelerobot commented Apr 1, 2024 via email

@namchuai namchuai moved this from In Progress to In Review in Menlo Apr 2, 2024
@namchuai namchuai moved this from In Review to QA in Menlo Apr 2, 2024
@namchuai
Copy link
Collaborator

namchuai commented Apr 3, 2024

I can't reproduce this issue after the fix. @Van-QA , please help us to re-validate. Thank you 🙏

@Van-QA
Copy link
Contributor

Van-QA commented Apr 8, 2024

As of now, on Windows even with Quick Ask OFF, closing Jan app (using the red X button) will not fully terminate all processes, and from that, users cannot reopen that Jan app.

Image

Jan v0.4.10-363
I will retry with Jan v0.4.10-364 and provide the result here. Updated: not facing the issue using Jan v0.4.10-364

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1: important Important feature / fix type: bug Something isn't working
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants