Skip to content

Add browser extension links to settings page#2159

Merged
droidmonkey merged 1 commit intokeepassxreboot:release/2.3.4from
varjolintu:browser_extension_link
Aug 20, 2018
Merged

Add browser extension links to settings page#2159
droidmonkey merged 1 commit intokeepassxreboot:release/2.3.4from
varjolintu:browser_extension_link

Conversation

@varjolintu
Copy link
Copy Markdown
Member

@varjolintu varjolintu commented Jul 27, 2018

Add browser extension links to browser integration settings page.

Description

Previously information about the new browser extension existed only in the FAQ or in the GitHub page. This makes the need of a browser extension more clear to the users.

Motivation and context

Fixes #2158.

How has this been tested?

Manually. Links open in the default browser.

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]

@TheZ3ro
Copy link
Copy Markdown
Contributor

TheZ3ro commented Aug 1, 2018

Instead of placing links to the various stores, what about creating a page like https://keepassxc.org/browser dedicated to the browser-extension project and link that instead?

@varjolintu
Copy link
Copy Markdown
Member Author

That could also work, yes.

@droidmonkey
Copy link
Copy Markdown
Member

I am going to approve it as-is for now, we can put together a "browser plugin" landing page for 2.4. This page should also include clear instructions and documentation for the whole process as well.

@droidmonkey droidmonkey merged commit 95a6008 into keepassxreboot:release/2.3.4 Aug 20, 2018
@varjolintu varjolintu deleted the browser_extension_link branch August 20, 2018 03:38
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.

3 participants