Skip to content

Re-implement AutoOpen functionality after refactor#2504

Merged
phoerious merged 2 commits intodevelopfrom
fix/autoopen
Nov 23, 2018
Merged

Re-implement AutoOpen functionality after refactor#2504
phoerious merged 2 commits intodevelopfrom
fix/autoopen

Conversation

@droidmonkey
Copy link
Copy Markdown
Member

Description

Adds AutoOpen functionality back in after the database widget(s) refactor. Adds capability to programmatically unlock a database from the database widget.

Motivation and context

Feature was removed after refactor.

How has this been tested?

Manually

Types of changes

  • ✅ Bug fix (non-breaking change which fixes an issue)

Checklist:

  • ✅ I have read the CONTRIBUTING document. [REQUIRED]
  • ✅ My code follows the code style of this project. [REQUIRED]
  • ✅ All new and existing tests passed. [REQUIRED]
  • ✅ I have compiled and verified my code with -DWITH_ASAN=ON. [REQUIRED]

@droidmonkey droidmonkey added bug pr: refactoring Pull request refactors code labels Nov 23, 2018
@phoerious
Copy link
Copy Markdown
Member

Two comments, but not of them critical.

@phoerious phoerious added this to the v2.4.0 milestone Nov 23, 2018
@droidmonkey
Copy link
Copy Markdown
Member Author

done

@phoerious phoerious merged commit ff7191e into develop Nov 23, 2018
@phoerious phoerious deleted the fix/autoopen branch November 23, 2018 18:25
@Emily7777
Copy link
Copy Markdown

Hi Guys. I can't seem to find where/how to use this feature in the latest KeepassXC (2.4.1). Where can I get more info on the master password -> auto open multi databases feature?

@droidmonkey
Copy link
Copy Markdown
Member Author

@Emily7777 please see #1053 (comment)

We do need to document this further...

@phoerious phoerious added pr: bugfix Pull request fixes a bug and removed bug labels Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: bugfix Pull request fixes a bug pr: refactoring Pull request refactors code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants