Feature/173197 persons api client nu get package (#578)

* Added a new step to the build workfow to build and publish a nuget package * Changed the target condition so it wont run in CI * Added a change for testing the new workflow * Added environment varibale CI to the CI yaml file * Added build-args in the docker-build * Fixed the casing in teh target condition * Modified target condition * Modified target condition * Added CI argument to the docker file, passed in from the docker build * Passed in CI argument to the project build process * Added CI argument to the docker build * Added CI argument to the build-and-push-image.yml file * Added changes to test Client Nuget Package deployment * added CI argumetn tot he build command in continious integration file * Added CI argument to the dotnet build in the dockerfile * Updated to test nuget deployment * Made changes to test nuget deployment --------- Co-authored-by: Farshad DASHTI <[email protected]>
DFE-Digital · Aug 14, 2024 · 8cdecac · 8cdecac
1 parent 240b6a6
commit 8cdecac
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/continuous-integration-dotnet.yml b/.github/workflows/continuous-integration-dotnet.yml
@@ -111,7 +111,7 @@ jobs:
         dotnet build -c Release
         dotnet pack -c Release --no-build --output .
         dotnet nuget push "*.nupkg" --api-key ${{ secrets.GITHUB_TOKEN }} --source https://nuget.pkg.github.com/DFE-Digital/index.json
-      if: steps.person-api-changes.client.src == 'true'
+      if: steps.person-api-changes.outputs.client == 'true'
 
     - name: Stop containers
       if: always()

diff --git a/Dfe.PersonsApi.Client/Generated/Client.g.cs b/Dfe.PersonsApi.Client/Generated/Client.g.cs
@@ -139,7 +139,7 @@ public virtual async System.Threading.Tasks.Task<MemberOfParliament> GetMemberOf
                         if (status_ == 404)
                         {
                             string responseText_ = ( response_.Content == null ) ? string.Empty : await response_.Content.ReadAsStringAsync().ConfigureAwait(false);
-                            throw new PersonsApiException("Constituency not found ", status_, responseText_, headers_, null);
+                            throw new PersonsApiException("Constituency not found", status_, responseText_, headers_, null);
                         }
                         else
                         if (status_ == 400)

diff --git a/Dfe.PersonsApi.Client/Generated/swagger.json b/Dfe.PersonsApi.Client/Generated/swagger.json
@@ -35,7 +35,7 @@
             }
           },
           "404": {
-            "description": "Constituency not found "
+            "description": "Constituency not found"
           },
           "400": {
             "description": "Constituency cannot be null or empty"

diff --git a/PersonsApi/Controllers/ConstituenciesController.cs b/PersonsApi/Controllers/ConstituenciesController.cs
@@ -20,7 +20,7 @@ public ConstituenciesController(IPersonsQueries personQueries)
         [HttpGet("{constituencyName}/mp")]
         [SwaggerOperation(Summary = "Retrieve Member of Parliament by constituency name", Description = "Receives a constituency name and returns a Person object representing the Member of Parliament.")]
         [SwaggerResponse(200, "A Person object representing the Member of Parliament.", typeof(MemberOfParliament))]
-        [SwaggerResponse(404, "Constituency not found ")]
+        [SwaggerResponse(404, "Constituency not found")]
         [SwaggerResponse(400, "Constituency cannot be null or empty")]
         public async Task<IActionResult> GetMemberOfParliamentByConstituencyAsync([FromRoute] string constituencyName, CancellationToken cancellationToken)
         {