From 3651f655e52a1bebc63cc8d909e67e41bd0c0f82 Mon Sep 17 00:00:00 2001 From: Paul Howes Date: Thu, 2 Apr 2020 10:15:29 +0100 Subject: [PATCH 01/35] CON-1492 Content change --- .../EmployerAgreementController.cs | 7 + .../Helpers/ControllerConstants.cs | 1 + .../Views/EmployerAgreement/Index.cshtml | 150 ++++++++---------- 3 files changed, 74 insertions(+), 84 deletions(-) diff --git a/src/SFA.DAS.EmployerAccounts.Web/Controllers/EmployerAgreementController.cs b/src/SFA.DAS.EmployerAccounts.Web/Controllers/EmployerAgreementController.cs index debb6b5104..dbf9aaabc7 100644 --- a/src/SFA.DAS.EmployerAccounts.Web/Controllers/EmployerAgreementController.cs +++ b/src/SFA.DAS.EmployerAccounts.Web/Controllers/EmployerAgreementController.cs @@ -327,5 +327,12 @@ public async Task WhenDoYouWantToView(int? choice, string agreemen } } } + + [HttpGet] + [Route("agreements/{accountLegalEntityHashedId}/viewallagreements")] + public ActionResult ViewAllAgreements(string hashedAccountId, string accountLegalEntityHashedId) + { + throw new NotImplementedException(); + } } } \ No newline at end of file diff --git a/src/SFA.DAS.EmployerAccounts.Web/Helpers/ControllerConstants.cs b/src/SFA.DAS.EmployerAccounts.Web/Helpers/ControllerConstants.cs index 66351a5189..b9288c50af 100644 --- a/src/SFA.DAS.EmployerAccounts.Web/Helpers/ControllerConstants.cs +++ b/src/SFA.DAS.EmployerAccounts.Web/Helpers/ControllerConstants.cs @@ -90,5 +90,6 @@ public static class ControllerConstants public const string ViewApprentice = "ViewApprentice"; public const string ApproveOrRejectApprentice = "ApproveOrRejectApprentice"; public const string ViewApprenticeBeforeApprove = "ViewApprenticeBeforeApprove"; + public const string ViewAllAgreementActionName = "ViewAllAgreements"; } } \ No newline at end of file diff --git a/src/SFA.DAS.EmployerAccounts.Web/Views/EmployerAgreement/Index.cshtml b/src/SFA.DAS.EmployerAccounts.Web/Views/EmployerAgreement/Index.cshtml index 3de5088281..7c9f499e32 100644 --- a/src/SFA.DAS.EmployerAccounts.Web/Views/EmployerAgreement/Index.cshtml +++ b/src/SFA.DAS.EmployerAccounts.Web/Views/EmployerAgreement/Index.cshtml @@ -15,110 +15,92 @@ } } -
-
-

Your organisations and agreements

-
-
-
- -
-
-
-
-
-

Add each organisation that will contract with your training providers.

-

There is no restriction on the number of connected organisations you can add.

-
-
-
-
-
- - Transfers status: - - @{ - var authorizationResult = Html.GetAuthorizationResult("EmployerFeature.TransferConnectionRequests"); - if (authorizationResult.IsAuthorized) - { - @:Enabled - } - else if (authorizationResult.HasError()) - { - @:Disabled - } - else - { - throw new ArgumentOutOfRangeException(); - } - } - - - -
-

You must sign the most up-to-date agreement for all organisations to use transfers functionality.

-
-
+@{Layout = "~/Views/Shared/_Layout_CDN.cshtml"; } + +
+
+

Your organisations and agreements

+

You must add each organisation that will contract with your training providers.

+

You can add as many connected organisations as you need to.

+

Transfers status:  + + @{ + var authorizationResult = Html.GetAuthorizationResult("EmployerFeature.TransferConnectionRequests"); + if (authorizationResult.IsAuthorized) + { + @:enabled. + } + else if (authorizationResult.HasError()) + { + @:disabled. + } + else + { + throw new ArgumentOutOfRangeException(); + } + } + +

+

+ Add an organisation +

-
-
- - - - - - - - +
+
+
OrganisationAgreement statusActionAgreement ID
+ + + + + + + - + @foreach (var employerAgreement in Model.Data.EmployerAgreementsData.EmployerAgreements) { - - - + + - - }
Organisations
Organisation nameLatest agreement statusActions
@employerAgreement.LegalEntity.Name +
@employerAgreement.LegalEntity.Name @if (employerAgreement.HasPendingAgreement) { - if (employerAgreement.HasSignedAgreement) - { - @:Updated agreement available - } - else - { - @:Agreement not accepted - } + @:Not yet accepted } else if (employerAgreement.HasSignedAgreement) { - @:Accepted by @employerAgreement.Signed.SignedByName + @:Accepted } - @if (employerAgreement.HasPendingAgreement) - { - Accept agreement - } - else if (employerAgreement.HasSignedAgreement) - { - View - } + + + View all agreements + - @if (employerAgreement.HasSignedAgreement) + + @if (Model.Data.EmployerAgreementsData.EmployerAgreements.Any()) { - @employerAgreement.LegalEntity.AccountLegalEntityPublicHashedId - } - else - { - Accept agreement to receive an agreement ID + Remove organisation }
- +
+ + + How transfers status is worked out + + +
+ Transfers are enabled if all the organisations in your account have accepted the latest agreement. +
+
+
+
+
+
@if (Model.Data.EmployerAgreementsData.EmployerAgreements.Any()) {

Remove an organisation from your account

From 7f7f78e6a68ac663a1d4010191ddfae688257f39 Mon Sep 17 00:00:00 2001 From: Paul Howes Date: Thu, 2 Apr 2020 12:44:04 +0100 Subject: [PATCH 02/35] Fixed breadcrumbs --- .../Views/EmployerAgreement/Index.cshtml | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/src/SFA.DAS.EmployerAccounts.Web/Views/EmployerAgreement/Index.cshtml b/src/SFA.DAS.EmployerAccounts.Web/Views/EmployerAgreement/Index.cshtml index 7c9f499e32..d6f772a49d 100644 --- a/src/SFA.DAS.EmployerAccounts.Web/Views/EmployerAgreement/Index.cshtml +++ b/src/SFA.DAS.EmployerAccounts.Web/Views/EmployerAgreement/Index.cshtml @@ -80,7 +80,7 @@ @if (Model.Data.EmployerAgreementsData.EmployerAgreements.Any()) { - Remove organisation + Remove organisation } @@ -109,10 +109,12 @@
@section breadcrumb { -