/QtPass

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

User robustness #33

Merged
merged 2 commits into from Apr 28, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@rdoeffinger @annejan
@rdoeffinger
Contributor

rdoeffinger commented Apr 26, 2015

Two patches that should make user editing a bit more robust.

rdoeffinger added some commits Apr 26, 2015

@rdoeffinger
Do not assume that all keys have subkeys. 
It is possible to create keys without sub-keys.
While that is a questionable setup, the case this code
is fixing is when the main key does not contain an identity
(relying exclusively on separate UIDs), which is at least as
questionable.
So try to make everyone happy by supporting both (though possibly
breaking yet another corner-case...).
add6638
@rdoeffinger
Ensure user dialog does not discard unknown keys. 
Instead display them at the top of the list.
4972a7f

annejan added a commit that referenced this pull request Apr 28, 2015

@annejan
Merge pull request #33 from rdoeffinger/user_robustness 
User robustness
beca5b9

@annejan annejan merged commit beca5b9 into IJHack:master Apr 28, 2015

@rdoeffinger rdoeffinger deleted the rdoeffinger:user_robustness branch May 2, 2015

annejan added a commit that referenced this pull request Jun 1, 2016

@annejan
Merge pull request #33 from rdoeffinger/user_robustness 
User robustness
53eaeea
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment