-
-
Notifications
You must be signed in to change notification settings - Fork 132
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
Add title when adding public keys #1171
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the PR!
Working on this, I realized that we should probably add more metadata like when it was added, when it was last used etc.
That's a great idea - happy to merge if you're willing to work on this.
passwords manager stopped recognizing the public id field as one.
What does this mean?
Oh - I wonder what logic it uses to recognize it. |
Not great logic tbh haha, since it thinks the title input is the ssh input on the https://github.com/settings/ssh/new page. |
@Eugeny The tests step seems to fail with the following trace:
Is this something I can fix? |
You can ignore the sonarqube error - however the migration is failing when there are existing keys in the database:
|
@Eugeny I fixed that (locally). However, I am running into issues now due to the name "label". In the user's view of the profile, that field was already being used for the abbreviated ssh key. See: ![]() ![]() What do you propose to change ? The name from label to back to title ? Or the old label to abbreviation? |
Let's change the old |
also fix db migration also fix warp-tech#1176
Looks good now - thanks! @all-contributors add @moalshak for code |
This PR adds the title input as a field in the frontend and saves it in the DB.
A view from the modal:
Public key after being saved:
Working on this, I realized that we should probably add more metadata like when it was added, when it was last used etc.
Fixes: #714
Fixes: #754
Fixes: #898
Fixes: #1176
One thing to note is that my passwords manager stopped recognizing the public id field as one.