Skip to content

add option to skip appName suffix in database folder path #646

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

Closed
wants to merge 1 commit into from

Conversation

richardliebmann
Copy link

add an option to skip the appName suffix in the electron database folder path

Pull request checklist

Please check if your PR fulfills the following requirements:

  • The changes have been tested successfully.

@robingenz
Copy link
Member

I close this as not planned for now. The documentation is also missing in this PR. Feel free to create a feature request to track the interest in the community. We might add this feature in the future.

@robingenz robingenz closed this Jun 17, 2025
@richardliebmann
Copy link
Author

Hi @robingenz,

The problem here is that the functional change is not possible at the moment because the library always modifies the database path. Do you have any other ideas on how we can proceed to solve this issue?

Thank you for your help!

@robingenz
Copy link
Member

@richardliebmann What do you mean by "functional change"?

@richardliebmann
Copy link
Author

Hi @robingenz,

The behaviour changed in a previous version, so an upgrade is not currently possible for us because the selected database location no longer works and would break our app. This change may have occurred around 2022, so it may only affect us and not other users. I hope that we can create a workaround so that we can upgrade without having to create a fork.

Thank you for your help!

@robingenz
Copy link
Member

@richardliebmann Please share the issue or commit where this change happened. I will take a look then.

@richardliebmann
Copy link
Author

Hi @robingenz,

It looks like my college has only created this behaviour in our fork. Sorry for bothering you! We will then integrate the solution into our fork and use it instead.

Thank you for your help!

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