Skip to content
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

Removed Duplicate CCLA Get Signature API Call #2862

Merged
merged 1 commit into from
Apr 9, 2021

Conversation

dealako
Copy link
Member

@dealako dealako commented Apr 8, 2021

  • Only load gerrit user list on approval list removals
  • Updated logging/error handling
  • Load both ICLA and CCLA gerrit users using a go routine (concurrently)
  • Resolved lint errors

Signed-off-by: David Deal [email protected]

@dealako dealako requested a review from wanyaland April 8, 2021 23:24
- Only load gerrit user list on approval list removals
- Updated logging/error handling
- Load both ICLA and CCLA gerrit users using a go routine (concurrently)
- Resolved lint errors
- Updated signature query by using the proper index

Signed-off-by: David Deal <[email protected]>
@dealako dealako force-pushed the bug/get-gerrit-users-debug branch from eb9cfdd to 1b833d2 Compare April 9, 2021 00:16
@dealako dealako merged commit 41e08cd into communitybridge:main Apr 9, 2021
@dealako dealako deleted the bug/get-gerrit-users-debug branch April 9, 2021 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant