Skip to content

Add custom icons when merging.#1008

Merged
louib merged 3 commits intokeepassxreboot:developfrom
louib:fix_merge_custom_icons
Oct 7, 2017
Merged

Add custom icons when merging.#1008
louib merged 3 commits intokeepassxreboot:developfrom
louib:fix_merge_custom_icons

Conversation

@louib
Copy link
Copy Markdown
Member

@louib louib commented Sep 29, 2017

Custom icons were not carried over when merging.

Also removed code related to autoEnableVisualHiding, since it's not used.

@droidmonkey I'd wait for the changes made in 2.2.1 to be carried over in develop before merging this. Do we have an ETA for that?

Motivation and context

Fixes #971

How has this been tested?

  • Unit test
  • 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]
  • ✅ I have added tests to cover my changes.

@louib louib added the bug label Sep 29, 2017
@louib louib added this to the v2.3.0 milestone Sep 29, 2017
@louib louib requested a review from droidmonkey September 29, 2017 18:10
@droidmonkey
Copy link
Copy Markdown
Member

Plan to mark 2.2.1 as final tonight.

@louib louib requested a review from TheZ3ro October 2, 2017 14:14
@louib louib force-pushed the fix_merge_custom_icons branch from a6a770b to 3905131 Compare October 2, 2017 14:15
@louib
Copy link
Copy Markdown
Member Author

louib commented Oct 2, 2017

@droidmonkey this one is ready for review now that 2.2.1 was upported!

@droidmonkey
Copy link
Copy Markdown
Member

Do we want to release this in 2.2.2?

@louib
Copy link
Copy Markdown
Member Author

louib commented Oct 7, 2017

@droidmonkey yeah maybe we could backport the fix itself to 2.2.2, but again with the merge function, upporting the tests if we change them in 2.2.2 is going to be a PITA, so I'd just backport the code in Database.cpp. Sounds good?

@louib louib force-pushed the fix_merge_custom_icons branch from 3905131 to a37a4d0 Compare October 7, 2017 14:28
louib added a commit to louib/keepassxc that referenced this pull request Oct 7, 2017
@louib louib merged commit 242faa1 into keepassxreboot:develop Oct 7, 2017
@louib louib deleted the fix_merge_custom_icons branch October 7, 2017 14:40
@droidmonkey
Copy link
Copy Markdown
Member

Yes sounds good

droidmonkey pushed a commit that referenced this pull request Oct 7, 2017
phoerious added a commit that referenced this pull request Oct 21, 2017
- Fixed entries with empty URLs being reported to KeePassHTTP clients [#1031]
- Fixed YubiKey detection and enabled CLI tool for AppImage binary [#1100]
- Added AppStream description [#1082]
- Improved TOTP compatibility and added new Base32 implementation [#1069]
- Fixed error handling when processing invalid cipher stream [#1099]
- Fixed double warning display when opening a database [#1037]
- Fixed unlocking databases with --pw-stdin [#1087]
- Added ability to override QT_PLUGIN_PATH environment variable for AppImages [#1079]
- Fixed transform seed not being regenerated when saving the database [#1068]
- Fixed only one YubiKey slot being polled [#1048]
- Corrected an issue with entry icons while merging [#1008]
- Corrected desktop and tray icons in Snap package [#1030]
- Fixed screen lock and Google fallback settings [#1029]
phoerious added a commit that referenced this pull request Feb 27, 2018
- Add support for KDBX 4.0, Argon2 and ChaCha20 [#148, #1179, #1230, #1494]
- Add SSH Agent feature [#1098, #1450, #1463]
- Add preview panel with details of the selected entry [#879, #1338]
- Add more and configurable columns to entry table and allow copying of values by double click [#1305]
- Add KeePassXC-Browser API as a replacement for KeePassHTTP [#608]
- Deprecate KeePassHTTP [#1392]
- Add support for Steam one-time passwords [#1206]
- Add support for multiple Auto-Type sequences for a single entry [#1390]
- Adjust YubiKey HMAC-SHA1 challenge-response key generation for KDBX 4.0 [#1060]
- Replace qHttp with cURL for website icon downloads [#1460]
- Remove lock file [#1231]
- Add option to create backup file before saving [#1385]
- Ask to save a generated password before closing the entry password generator [#1499]
- Resolve placeholders recursively [#1078]
- Add Auto-Type button to the toolbar [#1056]
- Improve window focus handling for Auto-Type dialogs [#1204, #1490]
- Auto-Type dialog and password generator can now be exited with ESC [#1252, #1412]
- Add optional dark tray icon [#1154]
- Add new "Unsafe saving" option to work around saving problems with file sync services [#1385]
- Add IBus support to AppImage and additional image formats to Windows builds [#1534, #1537]
- Add diceware password generator to CLI [#1406]
- Add --key-file option to CLI [#816, #824]
- Add DBus interface for opening and closing KeePassXC databases [#283]
- Add KDBX compression options to database settings [#1419]
- Discourage use of old fixed-length key files in favor of arbitrary files [#1326, #1327]
- Correct reference resolution in entry fields [#1486]
- Fix window state and recent databases not being remembered on exit [#1453]
- Correct history item generation when configuring TOTP for an entry [#1446]
- Correct multiple TOTP bugs [#1414]
- Automatic saving after every change is now a default [#279]
- Allow creation of new entries during search [#1398]
- Correct menu issues on macOS [#1335]
- Allow compilation on OpenBSD [#1328]
- Improve entry attachments view [#1139, #1298]
- Fix auto lock for Gnome and Xfce [#910, #1249]
- Don't remember key files in file dialogs when the setting is disabled [#1188]
- Improve database merging and conflict resolution [#807, #1165]
- Fix macOS pasteboard issues [#1202]
- Improve startup times on some platforms [#1205]
- Hide the notes field by default [#1124]
- Toggle main window by clicking tray icon with the middle mouse button [#992]
- Fix custom icons not copied over when databases are merged [#1008]
- Allow use of DEL key to delete entries [#914]
- Correct intermittent crash due to stale history items [#1527]
- Sanitize newline characters in title, username and URL fields [#1502]
- Reopen previously opened databases in correct order [#774]
- Use system's zxcvbn library if available [#701]
- Implement various i18n improvements [#690, #875, #1436]
@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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Merging database does not bring over custom icons

4 participants