Skip to content

Conversation

@smittal22
Copy link
Contributor

@smittal22 smittal22 commented Oct 17, 2023

Reason for Change:
e #25350573 on ADO

Issue Fixed:

Requirements:

Notes:

@smittal22 smittal22 requested a review from a team as a code owner October 17, 2023 19:27
@smittal22 smittal22 self-assigned this Oct 17, 2023
@smittal22 smittal22 force-pushed the change-return-code branch 3 times, most recently from 6113510 to 3b3df7f Compare October 20, 2023 18:39
@smittal22 smittal22 force-pushed the change-return-code branch 3 times, most recently from 30bb148 to 82f3887 Compare December 4, 2023 20:12
@smittal22 smittal22 requested a review from a team as a code owner December 7, 2023 18:47
@smittal22 smittal22 force-pushed the change-return-code branch 8 times, most recently from c8c2dbf to 172748e Compare December 11, 2023 20:30
@smittal22 smittal22 force-pushed the change-return-code branch 3 times, most recently from f3dd075 to 41c9d6d Compare December 12, 2023 21:23
@smittal22 smittal22 enabled auto-merge (squash) December 12, 2023 21:33
@smittal22 smittal22 merged commit 8798f10 into master Dec 13, 2023
@smittal22 smittal22 deleted the change-return-code branch December 13, 2023 20:05
paulyufan2 pushed a commit that referenced this pull request Dec 18, 2023
* change expected code to 200, since DNC returns 200

* Use http client interface instead of passing in a client directly

* Write test case

* Write test case

* Write test cases

* Write test cases

* Write test cases

* Write test cases

* Write test cases

* Write test cases

* Write test cases

* fix linter issues
matmerr pushed a commit that referenced this pull request Jan 17, 2024
* change expected code to 200, since DNC returns 200

* Use http client interface instead of passing in a client directly

* Write test case

* Write test case

* Write test cases

* Write test cases

* Write test cases

* Write test cases

* Write test cases

* Write test cases

* Write test cases

* fix linter issues
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.

5 participants