GPG verification doesn't load in files commit partial
Designs
- Show closed items
Relates to
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Author Maintainer
/cc @jschatz1
- Gabriel Mazetto added diff label
added diff label
- Gabriel Mazetto marked this issue as related to #20268 (closed)
marked this issue as related to #20268 (closed)
- Developer
@iamphill I can't find the place where you took the screenshot from. Can you tell me where it was?
- Author Maintainer
@winh I saw it on https://dev.gitlab.org/gitlab/gitlabhq/ but it'll only be there if the last commit is signed
- Developer
I'm setting up GPG and look into this.
- Winnie Hellmann assigned to @winh
assigned to @winh
- Winnie Hellmann mentioned in issue #35668 (moved)
mentioned in issue #35668 (moved)
- Winnie Hellmann added repository and removed bug diff labels
added repository and removed bug diff labels
- Winnie Hellmann mentioned in merge request !13308 (closed)
mentioned in merge request !13308 (closed)
- Developer
I need some backend help for writing a test. We currently have no factory to create a JSON response with signed commits. There seems to be no test for the controller of the JSON response (
app/controllers/projects/commits_controller.rb
). - Developer
This requires some more backend work: Currently the JSON endpoint loads the GPG signatures of all commits while we need only a single one for the repository page. So I think we should pass the commit SHA there.
Maybe it would be a good idea to just remove the GPG signature from the repository page altogether until the backend changes have been made.
@jschatz1 what do you think?
- Developer
To avoid being dependent on the backend, I have created a new issue for displaying the GPG status (https://gitlab.com/gitlab-org/gitlab-ce/issues/36046) and will focus here on hiding the loading indicator.
- Winnie Hellmann mentioned in merge request !13337
mentioned in merge request !13337
- Winnie Hellmann mentioned in commit 335ee51e
mentioned in commit 335ee51e
- Winnie Hellmann mentioned in commit 97f251de
mentioned in commit 97f251de
- Winnie Hellmann mentioned in merge request !13526 (merged)
mentioned in merge request !13526 (merged)
- Phil Hughes closed via merge request !13526 (merged)
closed via merge request !13526 (merged)
- Phil Hughes mentioned in commit 85d38bcd
mentioned in commit 85d38bcd