Skip to content

Provide warning to users with Qt 5.5.x about potential crash#2211

Merged
droidmonkey merged 1 commit intorelease/2.3.4from
fix/warn-qt55
Aug 20, 2018
Merged

Provide warning to users with Qt 5.5.x about potential crash#2211
droidmonkey merged 1 commit intorelease/2.3.4from
fix/warn-qt55

Conversation

@droidmonkey
Copy link
Copy Markdown
Member

Description

Warns users that crashes can occur if they are running KeePassXC on Qt 5.5. This will assist with reporting issues such as #335, #1254, and #1554.

Types of changes

  • ✅ Bug fix (non-breaking change which fixes an issue)
  • ✅ New feature (non-breaking change which adds functionality)
  • ✅ Breaking change (fix or feature that would cause existing functionality to change)

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 this to the v2.3.4 milestone Aug 18, 2018
@droidmonkey droidmonkey requested a review from phoerious August 18, 2018 14:36
@phoerious
Copy link
Copy Markdown
Member

There should be an option to disable the message.

@droidmonkey
Copy link
Copy Markdown
Member Author

Good point

@droidmonkey
Copy link
Copy Markdown
Member Author

I fixed a typo and wrapped it with a config setting that makes sure the warning is only shown once.

@droidmonkey droidmonkey merged commit bb73df5 into release/2.3.4 Aug 20, 2018
@droidmonkey droidmonkey deleted the fix/warn-qt55 branch August 20, 2018 03:39
droidmonkey added a commit that referenced this pull request Aug 22, 2018
- Show all URL schemes in entry view [#1768]
- Disable merge when database is locked [#1975]
- Fix intermittent crashes with favorite icon downloads [#1980]
- Provide potential crash warning to Qt 5.5.x users [#2211]
- Disable apply button when creating new entry/group to prevent data loss [#2204]
- Allow for 12 hour timeout to lock idle database [#2173]
- Multiple SSH Agent fixes [#1981, #2117]
- Multiple Browser Integration enhancements [#1993, #2003, #2055, #2116, #2159, #2174, #2185]
- Fix browser proxy application not closing properly [#2142]
- Add real names and Patreon supporters to about dialog [#2214]
- Add settings button to toolbar, Donate button, and Report a Bug button to help menu [#2214]
- Enhancements to release-tool to appsign intermediate build products [#2101]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants