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

Return error if create credential request is not a bad request #4874

Merged
merged 1 commit into from
Dec 21, 2022

Conversation

bjoginapally
Copy link
Contributor

@bjoginapally bjoginapally commented Dec 20, 2022

Description

Please explain the changes you've made.

Issue reference

Fixes: #issue_number

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Adds necessary unit tests for change
  • Adds necessary E2E tests for change
  • Unit tests passing
  • Extended the documentation / Created issue for it

@bjoginapally bjoginapally requested a review from a team as a code owner December 20, 2022 23:41
@bjoginapally bjoginapally force-pushed the bjoginapally/credential_controller branch from 9b75768 to 4d4a3b2 Compare December 20, 2022 23:42
@ytimocin
Copy link
Contributor

Do we need to add tests for this?

@bjoginapally bjoginapally merged commit 245bea0 into main Dec 21, 2022
@bjoginapally bjoginapally deleted the bjoginapally/credential_controller branch December 21, 2022 18:49
mishrapratikshya pushed a commit that referenced this pull request Feb 2, 2023
# Description

_Please explain the changes you've made._

## Issue reference

<!--
We strive to have all PR being opened based on an issue, where the
problem or feature have been discussed prior to implementation.
-->

Fixes: #issue_number

## Checklist

Please make sure you've completed the relevant tasks for this PR, out of
the following list:

* [x] Code compiles correctly
* [ ] Adds necessary unit tests for change
* [ ] Adds necessary E2E tests for change
* [x] Unit tests passing
* [ ] Extended the documentation / Created issue for it

Co-authored-by: Bharath Joginapally <[email protected]>
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.

3 participants