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

feat: allow user to move jan folder #1649

Merged
merged 3 commits into from
Jan 22, 2024
Merged

Conversation

namchuai
Copy link
Collaborator

@namchuai namchuai commented Jan 17, 2024

Describe Your Changes

Users can change default Jan app data location, give users control over how their own fs is used.

Flow:

  1. User enable Experimental Toggle from Jan > Settings > Advanced Settings
  2. Choose "Select" from Jan Data Folder settings
  3. Select folder to migrate Jan Root Directory
  4. App will restart itself automatically to reflect the changes
  5. See Jan Root Directory updated

Notice:

This is an experimental feature, users now have to enable Experimental Toggle from settings to access this feature until #1618 is implemented.
Docs will be updated after #1618 is implemented

Fixes Issues

Screenshots

image

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

@github-actions github-actions bot added the type: feature request A new feature label Jan 17, 2024
@namchuai namchuai force-pushed the feat/allow-user-move-jan-folder branch 7 times, most recently from 1f04ca4 to 73a52d3 Compare January 19, 2024 07:53
@namchuai namchuai requested a review from louis-menlo January 19, 2024 07:54
@namchuai namchuai marked this pull request as ready for review January 22, 2024 02:14
Signed-off-by: James <[email protected]>

chore: add get app path api to access from node module

update api name and handle some cases when syncing files

Signed-off-by: James <[email protected]>

remove redundant API

Signed-off-by: James <[email protected]>

add current path

Signed-off-by: James <[email protected]>
@louis-menlo louis-menlo force-pushed the feat/allow-user-move-jan-folder branch from a86b243 to 0610408 Compare January 22, 2024 06:23
@louis-menlo louis-menlo force-pushed the feat/allow-user-move-jan-folder branch 5 times, most recently from 5425521 to fbfd924 Compare January 22, 2024 07:20
Copy link
Contributor

@louis-menlo louis-menlo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@louis-menlo louis-menlo force-pushed the feat/allow-user-move-jan-folder branch from fbfd924 to 2f9cb2f Compare January 22, 2024 07:24
@namchuai namchuai merged commit 4cf4777 into dev Jan 22, 2024
7 checks passed
@namchuai namchuai deleted the feat/allow-user-move-jan-folder branch January 22, 2024 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants