Skip to content

Releases: SkillsFundingAgency/das-commitments

4.1.1038

17 Feb 13:42
772e65b
Compare
Choose a tag to compare

Changes:

  • 772e65b Merge pull request #666 from SkillsFundingAgency/CV-405-CV-510-CV-511-Notifications-Provider-Send-Approve-TransferSender-Approve-Reject
  • 1e6f97e Fixed template fields so they match the names in the templates
  • df9bdf9 Merge pull request #670 from SkillsFundingAgency/CV-639-fix-user-id-missing
  • f84e0ce Included apprentices when apprioving cohort
  • dc4471e Merge branch 'CV-405-CV-510-CV-511-Notifications-Provider-Send-Approve-TransferSender-Approve-Reject' into CV-639-fix-user-id-missing
  • 174edf8 Fixed stub issue
  • 8d1031f added UserId to API
  • c59891c Merge pull request #668 from SkillsFundingAgency/MF-475_fix_provider_search_for_first_name_last_search
  • 7cf6000 Merge pull request #659 from SkillsFundingAgency/MF-409-search
  • 66aa90d Merge branch 'master' into CV-405-CV-510-CV-511-Notifications-Provider-Send-Approve-TransferSender-Approve-Reject
See More
  • 265964a New indexes for search
  • ac2d2dc Changes so that search matches name
  • 05657bf renamed method
  • c1131fe Merge branch 'master' into CV-528-move-legacy-events-into-v2-handler
  • 6254205 added unit tests for tracking entity change
  • 6ed2a8b added tracking to transferRequestedRejected
  • 8c17370 Merge branch 'CV-528-move-transfer-sender-rejection-into-v2' into CV-528-move-legacy-events-into-v2-handler
  • a844d13 Merge branch 'CV-405-CV-510-CV-511-Notifications-Provider-Send-Approve-TransferSender-Approve-Reject' into CV-528-move-transfer-sender-rejection-into-v2
  • a2ab963 Merge branch 'master' into CV-405-CV-510-CV-511-Notifications-Provider-Send-Approve-TransferSender-Approve-Reject
  • 631b90c removed unused using statement
  • 2dc8896 moved legacy publish from command handlers to event handlers
  • 8256337 Created handler for Legacy event CohortApprovalRequestedByProvider
  • 37fab21 Merge branch 'master' of github.com:SkillsFundingAgency/das-commitments into MF-409-search
  • d67ab0f changed TransferRequestCreatedEventHandler to send legacy event for employer and provider
  • f66183a Unit test name refactoring
  • 7e7d1c7 moved legacy event 'CohortRejectedByTransferSender' from API into V2 message handler
  • 086234b moved legacy event CohortApprovedByTransferSender into V2 MessageHandler
  • 59d525b CV-528 Reject handler tests + Implementation
  • c33983b CV-528 Updated Event domain object
  • 2b0b21f Merge branch 'CV-405-CV-510-CV-511-Notifications-Provider-Send-Approve-TransferSender-Approve-Reject' into CV-528-move-transfer-sender-rejection-into-v2
  • 5f72d75 Merge pull request #662 from SkillsFundingAgency/CV-619-TransferSender-Emails
  • 67d7fa9 Merge branch 'CV-405-CV-510-CV-511-Notifications-Provider-Send-Approve-TransferSender-Approve-Reject' into CV-528-move-transfer-sender-rejection-into-v2
  • 673a8e8 Adds Transfer Rejection notifications
  • aa40a09 added event handler to consume the TransferRequestedRejectedEvent
  • 6552516 MF-409 merging in latest MF-427
  • 4e24abe Copy of notifications templates
  • b8cd343 MF-409 added missing unit test.
  • dfcef82 Transfer Sender Approval event handler for email notifications to employer and provider
  • d9ebd38 Added the transferRejectedEvent and added the RejectTransferRequestCommand to the message test harness
  • b9a1bb0 added unit tests for Approve/reject commands
  • 5bb2095 Adds SendProviderEmailCommand, handler, and tests
  • aee01c1 MF-409 merging in MF-423.
  • d50c20b MF-409 modified search to be better formed sql.
  • 6c591ec MF-409 added search term lookup to filtering apprenticeships.
  • 349c782 Merge branch 'CV-511-move-transfer-sender-approval-into-v2' into CV-528-move-transfer-sender-rejection-into-v2
  • 44eddae intial V1 message sent
  • 7e5caaa Merge branch 'CV-511-move-transfer-sender-approval-into-v2' into CV-405-CV-510-CV-511-Notifications-Provider-Send-Approve-TransferSender-Approve-Reject
  • d39150f PR coments
  • 240eb57 Merge branch 'CV-510-Provider-Cohort-Approval-Rewire' of https://github.com/SkillsFundingAgency/das-commitments into CV-510-Provider-Cohort-Approval-Rewire
  • 5138636 Addresses tech review points
  • 84de014 Merge branch 'MF-423_ApprenticeSearchOptimisation' of github.com:SkillsFundingAgency/das-commitments into MF-409-search
  • bf06e71 CV-528 RejectTransferRequestCommand NSB
  • 2601c58 Merge branch 'CV-405-SendEmailToEmployerCommandHandler' into CV-511-move-transfer-sender-approval-into-v2
  • ca8c2f6 Merge branch 'CV-405-SendEmailToEmployerCommandHandler' into CV-510-Provider-Cohort-Approval-Rewire
  • 498eb76 Merge branch 'master' into CV-510-Provider-Cohort-Approval-Rewire
  • 8984ca8 Merge branch 'master' into CV-405-SendEmailToEmployerCommandHandler
  • 865c154 Merge branch 'master' into CV-511-move-transfer-sender-approval-into-v2
  • eb654cd removed old code and wired up the new message sender
  • 4d708bd Merge branch 'MF-423_ApprenticeSearchOptimisation' of github.com:SkillsFundingAgency/das-commitments into MF-409-search
  • 99ae9d3 MF-409 mapping search term to handler.
  • c4600ee removed whitespace
  • f12e4e7 renamed test classes
  • 7aa67f4 MF-409 adding search term to api client.
  • f3f4851 reworked Message Handler to call Cohort.Approve() directly
  • 80ec106 reversed code to move the authenticationService into CohortController
  • 3e90fb3 chnaged style of tests
  • 1a48772 added Tests for TransferRequestApprovedEventHandler
  • d768970 removed AuthenticationService from DomainService for ApproveCohort
  • a6a71cd Merge branch 'MF-427_GetFilteredCsvDownload' of github.com:SkillsFundingAgency/das-commitments into MF-409-search
  • f2b968d added a new event handler to consume teh Event TransferRequestApproved
  • 8e1f673 Template url tweaks
  • 0a345f7 Merge branch 'CV-405-SendEmailToEmployerCommandHandler' into CV-510-Provider-Cohort-Approval-Rewire
  • c52fcd9 Another fix
  • 78e605a Fixes bugs
  • 7dbdceb Rewires Provider Send into v2
  • 417a6b1 Adds EmployerCohortApproved notification send
  • 59944aa Merge branch 'CV-405-SendEmailToEmployerCommandHandler' into CV-511-move-transfer-sender-approval-into-v2
  • 0b7e392 Merge branch 'master' into CV-405-SendEmailToEmployerCommandHandler
  • 2eaa965 basic handler for processing TransferSenderApproveCommand
  • b318051 Merge branch 'master' into CV-510-Provider-Cohort-Approval-Rewire
  • 60a9787 Adds "EmployerTransferPendingFinalApproval" email send
  • 1745e2f Test correction
  • dccdea6 Fixes tokens
  • 74093d2 MF-409 added search term parameter.
  • c41e9e1 Modifies event handler so that only provider->employer notification is sent
  • 4630d07 Tokens for email notification
  • 0835da0 Test cohort approval
  • d1b1f63 Adds new test for v2 command sending
  • dbe21ff Removes obsolete tests
  • 403ff4d Merge branch 'CV-405-SendEmailToEmployerCommandHandler' into POC_Provider_Approve_Cohort_Rewire
  • d274efe Adds user info
  • 0673f73 Merge branch 'master' into POC_Provider_Approve_Cohort_Rewire
  • 22b072f uninstalled and re-installed SlowCheetah
  • 9528d03 Merge branch 'master' into CV-405-SendEmailToEmployerCommandHandler...
Read more

4.1.992

11 Feb 10:45
1b20f11
Compare
Choose a tag to compare

Changes:

  • 1b20f11 Merge pull request #653 from SkillsFundingAgency/CV-565-Use-Cohort-AccountLegalEntityId
  • 7d30b8d Merge branch 'CV-627-Improve-NSB-Logging-Around-Key-Events' into CV-565-Use-Cohort-AccountLegalEntityId
  • e97fa7e Adds extra logging in key command handler
  • ff691c7 Merge branch 'master' into CV-565-Use-Cohort-AccountLegalEntityId
  • 64c9390 Merge pull request #618 from SkillsFundingAgency/CV-106-Remove-End-Date-Validation-In-Past
  • 227f883 Merge branch 'master' into CV-565-Use-Cohort-AccountLegalEntityId
  • 8353209 Merge branch 'master' into CV-106-Remove-End-Date-Validation-In-Past
  • a655e6e Merge pull request #654 from SkillsFundingAgency/MF-427_GetFilteredCsvDownload
  • bf9d371 MF-427 fixed merge issues.
  • eecd351 MF-427 merging in master.
See More
  • e211880 Merge pull request #661 from SkillsFundingAgency/AppenticeStatusChange
  • afd17ed Change so that Filter Values are cached
  • e25b2d3 Add CacheRegistry to IOC
  • b518b7e Add cache storage service
  • 8ce7632 Correct Mapping
  • 5e0a9d2 Change to use contains
  • e284f01 Changes status to support enum types
  • 6d39ee3 Merge pull request #656 from SkillsFundingAgency/MF-423_ApprenticeSearchOptimisation
  • b7cd828 Move index to correct table definition
  • ebc51e7 Remove status from filter results
  • a6d48d6 Add missing indexes for filter values
  • 60f7a67 MF-423 fix line endings.
  • 94053d5 Fixed issue with paging and duplicates
  • bf27078 Merge branch 'master' into CV-106-Remove-End-Date-Validation-In-Past
  • 7f99351 Merge branch 'master' into CV-565-Use-Cohort-AccountLegalEntityId
  • 625a987 Merge branch 'MF-427_GetFilteredCsvDownload' into MF-423_ApprenticeSearchOptimisation
  • 162cdf2 Changed logic for getting apprentices to make it more efficient
  • 9cb35c1 MF-427 merge in MF-429.
  • 8eba84e Merge branch 'master' into CV-565-Use-Cohort-AccountLegalEntityId
  • 62a2536 Merge branch 'master' of github.com:SkillsFundingAgency/das-commitments into MF-427_GetFilteredCsvDownload
  • 2074adb fixing stuff lost in merge
  • a963e73 Merge fixes
  • eee2058 merging
  • b48a436 merging
  • 2be9dd9 Merge branch 'manage_apprentices' into MF-427_GetFilteredCsvDownload
  • c7cd090 Merge branch 'manage_apprentices' into MF-427_GetFilteredCsvDownload
  • 2cd159c Fixed merge issues
  • 7f004a7 Merge branch 'manage_apprentices' into MF-427_GetFilteredCsvDownload
  • 5644e4d Missing merged code readded
  • fd031ba Merge branch 'manage_apprentices' into MF-427_GetFilteredCsvDownload
  • 7512fde fixed unit tests intermittenly failing
  • 27463f6 Fixed broken test
  • ce1c369 Update mapper to map ApprenticeshipStatus
  • 4228932 Change mapper to be an extension
  • 67f7625 Merge branch 'manage_apprentices' :
  • 335c5fe Wired up total apprentices available value
  • a52b96c Fixed merge issues
  • 15037c4 fixed merge markers
  • dc60f23 merging Manage into branch
  • 71bf30b Merge branch 'manage_apprentices' into MF-427_GetFilteredCsvDownload
  • 4cba866 Changed date format to include daylight saving adjustments
  • f6248e1 Made start and end date filter handling more robust
  • 52112b7 Fixed failing test
  • 41c907c Adding filter support for client
  • 216df17 Merge branch 'manage_apprentices' into MF-429-Map_Payment_Status_To_Apprenticeship_Status
  • f4d48d7 Changed 'finished' to 'completed' on ApprenticeshipStatus at PO request
  • e07e624 Removes redundant script
  • 9c0b5dd Tidy up
  • b757bc9 Fixes broken tests
  • 8e25257 Merge branch 'master' into CV-565-Use-Cohort-AccountLegalEntityId
  • da43571 Added filter support to get apprentice queries
  • 76e6b8e changed response type of ApprenticeshipStuatus to string for formatting
  • ce04b54 Created extension for filtering apprenticeships
  • 807fda4 Moved ApprenticeshipStatus to base class and set configuration to ignore it
  • d1ec680 reverted odd changes from previous commits
  • a1572c6 Added ApprenticeshipStatus to response
  • 4a1c9c3 Added mapper for ApprenticeshipStatus from PaymentStatus. Modified sorting to use ApprenticeshipStatus instead of PayMentStatus. Fixed handler unit tests broken by adding mapper and using ApprenticeshipStatus for sorting. Added unit tests for ApprenticeshipStatus mapper.
  • acab7a7 Wired up filter value api call to api client
  • 7bf3c25 Separated handler tests into distinct areas
  • a20fa07 Added Api controller code for accepting filter values
  • 966b4fb Merge branch 'master' into CV-106-Remove-End-Date-Validation-In-Past
  • 864e2a9 Merge branch 'master' into CV-565-Use-Cohort-AccountLegalEntityId
  • 06a7766 Merge branch 'master' into CV-565-Use-Cohort-AccountLegalEntityId
  • 812bbb1 Adds TransferSender -> Cohort relationship
  • cd770c0 Adds AccountLegalEntityId to approval event
  • b8da06c GetCohortSummary refactoring
  • 1466dab Swapped post- for pre- deployment scripts
  • f5d060e Adds data script to post-deployment scripts
  • deda56d CV-106
  • cb231ae CV-106
  • 209f500 CV-106
  • 84188d5 Merge branch 'CV-549-Data-Overflow' into CV-106-Remove-End-Date-Validation-In-Past
  • 1dc49b3 Merge branch 'master' into CV-106-Remove-End-Date-Validation-In-Past
  • 6cad230 CV-106
  • 99f457c Removes validation rule

This list of changes was auto generated.

4.1.896

30 Jan 14:05
df1f30f
Compare
Choose a tag to compare

Changes:

  • df1f30f Merge pull request #652 from SkillsFundingAgency/CV-571-Apprenticeship-Details-v2
  • dd31079 Merge branch 'master' into CV-571-Apprenticeship-Details-v2
  • e2f012a Merge pull request #629 from SkillsFundingAgency/manage_apprentices
  • 6025624 CV-571 Decimal Extension test
  • 5f9acca CV-571 Enum Description Test
  • fb60ce1 CV-571 Unit test
  • 2cabe15 Merge remote-tracking branch 'origin/master' into manage_apprentices
  • cf64199 Nesting of ApprenticeshipDetails as per CR
  • a97cf91 Update mapper
  • 5cd542c Merge branch 'master' into CV-571-Apprenticeship-Details-v2
See More
  • e3ea8d1 Merge remote-tracking branch 'origin/manage_apprentices' into manage_apprentices
  • 21329c0 Added mapper to map apprenticeship details to response
  • 40e05c2 Rename of Mediator result class
  • 9cd55a1 Move controller action
  • c18d3a5 Rename GetApprenticeshipRequest to Query
  • 1da7c64 Change alerts mapper to extension
  • 1c4e1ba Remove incldue statements not required
  • db28ec2 Correct issue from merge
  • 1192a59 Merge remote-tracking branch 'origin/master'
  • f0a9700 Add missing include for ApprenticeshipUpdate table
  • 54f1fa4 Update SQL include
  • f9b961c Change so that Resolved alerts arent mapped
  • d9d971f Merge pull request #651 from SkillsFundingAgency/MF-424-Remove_PendingUpdateOriginator_Usage
  • 67a4459 Add correct table mapping
  • 35cab67 Map Apprenticeship Update table
  • db473c0 Cover null for ApprenticeshipUpdate
  • 82773a9 Change alert mapping for ApprenticeshipUpdate
  • 3853a7f Adds decimal extensions class
  • e05efa1 Remove pendingupdateoriginator from alerts mapper
  • 561027f Add apprenticeship update EF mapping
  • 4ac9a71 Remove TODO
  • 1f25f1f Adds enum extension for getting desc attr
  • 279c748 Adds Description attr to type
  • 75464db Adds Price Episodes to api client
  • d568444 Merge remote-tracking branch 'origin/manage_apprentices' into MF-424-Remove_PendingUpdateOriginator_Usage
  • 9db7777 Replaced PendingUpdateOriginator with check against DataLock table, fixed unit tests affected by this change. Check against ApprenticeshipUpdate table to do
  • d968144 Merge pull request #630 from SkillsFundingAgency/MF-374-Inital_Apprentice_Sort_for_manage_your_apprenticeships
  • 30304a5 Merge branch 'CV-555-ConfirmEmployer' into CV-571-Apprenticeship-Details-v2
  • c4c009d Undid inadvertent change to v1 type :-/
  • 1723702 Adds new client method for getting apprenticeship details
  • ae60bb5 Swaps out PaymentStatus in favour of new ApprenticeshipStatus
  • c311527 MF-374 fixing missing map of apprenticeship id.
  • 1e6c78e Merge branch 'MF-374-Inital_Apprentice_Sort_for_manage_your_apprenticeships' of https://github.com/SkillsFundingAgency/das-commitments into MF-374-Inital_Apprentice_Sort_for_manage_your_apprenticeships
  • 1c1d016 added logic to allow reverse sorting by alerts logic to be reached and associated unit test
  • 95970a0 Remove uint for provider id
  • 22ac877 Merge remote-tracking branch 'origin/master' into MF-374-Inital_Apprentice_Sort_for_manage_your_apprenticeships
  • f949bd4 Remove dot net core dependency
  • 8d4dd8d Merge remote-tracking branch 'origin/MF-375_Add_Apprenticeship_Paging' into MF-374-Inital_Apprentice_Sort_for_manage_your_apprenticeships
  • 00b67be Changes provider id to long for consistency
  • 0bb9607 Merge remote-tracking branch 'MF-375'
  • b1d00e8 Code review changes
  • 90a7ed0 Merge branch 'manage_apprentices' into MF-374-Inital_Apprentice_Sort_for_manage_your_apprenticeships
  • 9001e75 Correction to where clause for filters
  • ea30b98 Merge branch 'MF-377_Add_Alerts_Total' into MF-375_Add_Apprenticeship_Paging
  • affcc02 Small code clean up of tests
  • b9dcdf4 Added total apprentices with alert count to response
  • fefc487 Rename Db Connection string name
  • 7e9ce67 Add Readonly Connection string
  • 0be0723 Moved the API response to the correct project
  • 156c35e Added support for total number of apprenticeships found
  • d8b8b7a Changed the paging logic to be optional
  • 4a26b08 Merge branch 'MF-374-Inital_Apprentice_Sort_for_manage_your_apprenticeships' of github.com:SkillsFundingAgency/das-commitments into MF-374-Inital_Apprentice_Sort_for_manage_your_apprenticeships
  • 167c2d6 MF-374 added prop to api type nuget as well.
  • 659956d fixed broken unit tests
  • 422e0ac Merge branch 'MF-374-Inital_Apprentice_Sort_for_manage_your_apprenticeships' into MF-375_Add_Apprenticeship_Paging
  • 1fa7f85 changed client to map reverseSortand unit test
  • 7aa23ea Merge branch 'MF-374-Inital_Apprentice_Sort_for_manage_your_apprenticeships' of github.com:SkillsFundingAgency/das-commitments into MF-374-Inital_Apprentice_Sort_for_manage_your_apprenticeships
  • 04ae340 MF-373 adding id for link on manage page.
  • cb5fc45 Added client parameters to capture page management
  • 5ec6a5d Merge branch 'MF-374-Inital_Apprentice_Sort_for_manage_your_apprenticeships' into MF-375_Add_Apprenticeship_Paging
  • 9c1c4a7 fixing merge conflicts
  • 5c0992a Fixed slight bug in sorting logic and added unit tests for reverse sorting
  • 95d0390 Merge branch 'MF-374-Inital_Apprentice_Sort_for_manage_your_apprenticeships' into MF-375_Add_Apprenticeship_Paging
  • 44a0924 Remove unused using statement
  • 09d7e0e Sort by alerts
  • 49fa868 modified client to allow for reverse sorting
  • ac2c742 Modified GetApprenticeshipRequest and associated handler to allow for reverse sorting
  • a4715e6 Added logic to handle paging requests for apprenticeships
  • 42b1592 Changed StopDate to EndDate to fix sorting issue and modified unit test to reflect change
  • d21ebea Added missing sort unit tests
  • 24be298 Added unit test for default sort oredring
  • 971a4d3 Merge branch 'MF-372_manage_apprentices'
  • 70ac7a4 Correct mapping for DataLockStatus
  • cd11049 Update client to take sort field
  • d9def46 Add ignore to WIP
  • 365c4cb Add validation for the sort field
  • 7b46912 Change sort on Apprenticeship
  • 7e83f33 Merge branch 'manage_apprentices' into MF-374-Inital_Apprentice_Sort_for_manage_your_apprenticeships
  • 3bcd4d6 Merge pull request #631 from SkillsFundingAgency/MF-373-manage-apprenticeships-alerts
  • 23a4968 MF-373 alerts added in apprenticeship mapper.
  • 7527dc5 MF-373 alert mapper created and tested.
  • 6df5547 MF-405 alerts for ilr and changes pending handled in alert mapper.
  • 5f41117 Renamed GetApprenticeships method in client
  • 60abf5b Slight name refacotring to remove Approved and readded lost sort functionality when rebasing
  • 4df38f4 fixed stuff from merging feature branch
  • 7ad2a3f Started adding unit tests
  • 4f43c4e Added methods to sort approved apprenticeships into initial list
  • 8511083 Merge pull request #625 from SkillsFundingAgency/MF-405-db-query-apprenticeships
  • 4c7590a Changed sort to be much more sensible and added unit tests
  • 5d...
Read more

4.1.851

23 Jan 16:21
48fd481
Compare
Choose a tag to compare

Changes:

  • 48fd481 Merge pull request #621 from SkillsFundingAgency/CV-517-Get-Approved-Cohorts
  • 5e436de Merge branch 'master' into CV-517-Get-Approved-Cohorts
  • 2368940 Merge branch 'master' into CV-517-Get-Approved-Cohorts
  • bf534a1 Merge branch 'master' into CV-517-Get-Approved-Cohorts
  • e314fb2 fixed a typo
  • b148094 Merge branch 'CV-498-Api-Endpoint-For-MA' into CV-517-Get-Approved-Cohorts
  • ebad5a3 CV-517
  • 98c69a9 CV-571
  • fa3ec48 CV-571
  • ed1a61e Get approved providers
See More
  • 93cd39a missed files
  • f21d686 Merge branch 'CV-498-Api-Endpoint-For-MA' into CV-571-Get-Approved-Cohorts
  • a78e39d CV-571

This list of changes was auto generated.

4.1.797: Merge pull request #638 from SkillsFundingAgency/CV-555-ConfirmEmployer

20 Jan 15:07
e4de05a
Compare
Choose a tag to compare

Changes:

  • e4de05a Merge pull request #638 from SkillsFundingAgency/CV-555-ConfirmEmployer
  • a7128f4 Merge branch 'master' into CV-555-ConfirmEmployer
  • 66f4284 Merge pull request #645 from SkillsFundingAgency/CV-549-Data-Overflow-with-CV-566-567-568-new-api-endpoints-for-testing
  • b599845 Makes AleId nullable until we back-fill all data
  • 9fc0400 Merge branch 'master' into CV-555-ConfirmEmployer
  • 153df90 Merge branch 'master' into CV-549-Data-Overflow-with-CV-566-567-568-new-api-endpoints-for-testing
  • dda670a Merge branch 'master' into CV-549-Data-Overflow-with-CV-566-567-568-new-api-endpoints-for-testing
  • 2ea3a45 Merge branch 'master' into CV-555-ConfirmEmployer
  • 5992534 CV-555
  • 784dff5 Merge branch 'master' into CV-549-Data-Overflow-with-CV-566-567-568-new-api-endpoints-for-testing
See More
  • bdba459 Merge branch 'CV-566-567-568-combined-for-testing' into CV-549-Data-Overflow-with-CV-566-567-568-new-api-endpoints-for-testing
  • 8c73ded Merge branch 'CV-555-ConfirmEmployer' of https://github.com/SkillsFundingAgency/das-commitments into CV-555-ConfirmEmployer
  • 838f62b CV-555
  • 777b4b5 Fixes issue where new column wasn't in view :-/
  • 230f0d6 Merge branch 'CV-567-move-apprenticeship-update-api-endpoint-to-v2' into CV-566-567-568-combined-for-testing
  • 881d92d added notfound response when no pending updates
  • b506754 removed some refs and added 404 response to pending update
  • a57fe21 Merge branch 'CV-568-Add-Apprenticeship-To-Api-V2' into CV-566-567-568-combined-for-testing
  • bb3e0cc renamed routes after, PR comments
  • d5372bd renamed Party to OriginatingParty
  • 678eafa PR changes, flattened out response types and mapped originator to Party. Also reworked the OldMapperTester to support the new shared IMapper class
  • ac0f77c CV-555
  • fca228f CV-555
  • 1323118 fix configuration issues on SQL table
  • ca2aae5 CV-555
  • cfb2aa2 added tests for ApprenticeUpdatecontroller
  • c4c8c5b CV-555
  • faefc7e Merge branch 'CV-555-ConfirmEmployer' of https://github.com/SkillsFundingAgency/das-commitments into CV-555-ConfirmEmployer
  • 85bb3d3 CV-555 More tests
  • 449f9e7 Controller implementation
  • 5f47306 added query tests for apprenticeshipUpdate
  • f886cee Api type and mapper
  • 0f7da5f Query handler implementation
  • 58eed8d renamed IMapper to IOldMapper and referenced project instead of pacakge plus added GetApprenticeshipQuery handler
  • ace83f2 Merge pull request #636 from SkillsFundingAgency/CV-521-Add-LegalEntityId-To-Cohort
  • 1b8bf1e CV-555
  • 86f0ce4 fixed names after merge
  • 9ad5aa4 Merge branch 'CV-566-Add-Price-Episodes-To-Api-V2' into CV-567-move-apprenticeship-update-api-endpoint-to-v2
  • 8c31988 added tests for CanAccessCohort
  • 039848e added access-apprenticeship endpoint
  • 046aa4a WIP query and handler
  • 2ac71ef More refactor/renaming
  • fc43a63 Adjusted names of other query classes for consistency
  • 4162478 Tidy up
  • 64818ed Adds controller implementation
  • 25fd8f6 QueryResult -> Response mapper
  • a7a9ca8 Handler implementation
  • fbbd762 Adds query classes WIP
  • e549824 Adds AccountLegalEntityId property to model and sets it in ctor
  • 1a47e00 Merge branch 'CV-512-Add-stub-relationships-client' into CV-555-ConfirmEmployer
  • c0717dd CV-555
  • 0507a2f CV-555
  • 4ec8d2b CV-555
  • de00b11 Merge branch 'master' into CV-512-Add-stub-relationships-client
  • e78b291 Adds stubbed api client
  • 3a8c1c4 Merge branch 'master' into CV-549-Data-Overflow
  • ba73111 CV-549

This list of changes was auto generated.

4.1.766

14 Jan 16:08
81a35e5
Compare
Choose a tag to compare

Changes:

  • 81a35e5 Merge pull request #619 from SkillsFundingAgency/CV-487-With-Transfer-Sender
  • e4de35e Merge branch 'master' into CV-487-With-Transfer-Sender
  • 7283a32 Merge pull request #649 from SkillsFundingAgency/CV-585-Http-Error
  • 052793e Fixes breaking http update
  • 930ba77 fix test subscriber, so it processes the messages correctly
  • a45e67b Merge branch 'CV-578_RaceConditionBug' into CV-487-With-Transfer-Sender
  • a3d1876 fix test so it uses a random DB name, so it doesn;t clash
  • 1a80e13 Merge branch 'CV-562-fix-healthcheck' into CV-487-With-Transfer-Sender
  • 9bb49d7 Merge branch 'master' into CV-487-With-Transfer-Sender
  • 690fb55 Changes to Azure Template in Preparation for ASE Migration. (#641)
See More
  • fd1237b CV-578 Null check to fix failing test
  • e7e95ec CV-578 Copied events list to variable
  • ecde300 Merge branch 'master' into CV-487-With-Transfer-Sender
  • 4e02a01 removed param
  • f753d23 removed space
  • f37fa14 removed ref to reservations API client
  • e648bfb Merge pull request #614 from SkillsFundingAgency/CV-508-Cease-PaymentOrder-Calculation
  • a3b7468 Merge branch 'master' into CV-487-With-Transfer-Sender
  • b0fab79 Merge pull request #627 from SkillsFundingAgency/CV-560-Modify-Stop-Script
  • 4b7a78c Removes deprecated scripts
  • e6523e8 Removes events api sql
  • fd72d6e Merge pull request #607 from SkillsFundingAgency/CV-498-Api-Endpoint-For-MA
  • e2a44d5 Merge pull request #620 from SkillsFundingAgency/CV-545-Add-Recommended-Indexes
  • 84fae5d Merge pull request #615 from SkillsFundingAgency/CV-458-Deleting-Last-DraftApprentice-Deletes-Cohort
  • 82a0aa9 Merge pull request #623 from SkillsFundingAgency/CON-1027-package-upgrade
  • 77c8c8a CON-1027-upgrade-client-to-support-migration-to-azure-ad
  • eec3ece Merge branch 'master' into CV-498-Api-Endpoint-For-MA
  • 584b169 Removes unused indexes
  • ba25bcd Adds indexes
  • 8e83b1e CV-487
  • 7202eed Merge branch 'master' into CV-487-With-Transfer-Sender
  • b1d76ba Merge branch 'master' into CV-458-Deleting-Last-DraftApprentice-Deletes-Cohort
  • 816a5e2 renamed variables
  • 0d75f9a test /refactoring
  • fbfb529 Changed query to help with the performance.
  • b069fbb Added index recommended by Azure.
  • ea8ac46 Added new columns to index include based on Azure recommendation.
  • c67532e Cohort self-destruct on emptying
  • cf6c6c6 Merge branch 'master' into CV-498-Api-Endpoint-For-MA
  • 3424ffa Removes re-order proc and all calls to it
  • 555b4ba Merge branch 'master' into CV-498-Api-Endpoint-For-MA
  • e4aa88a Addressing PR comments
  • d7b7d53 Removes whitespace :-|
  • c20f181 Adds Client method and test
  • 4fd2d5b Handler tests
  • 63c80e0 Adds validator test
  • 33373b7 AccountController implementation and test
  • e3c5ced Moved to a two-query approach for readability
  • e240d77 Added PredicateBuilder
  • 5ba2272 added Message type which contains message text and datetime sent
  • 28da7c1 fix broken test
  • 607cc61 added ToListAsync
  • c01142f removed whitespace
  • 4d983ab added unit tests for GetCohorts query
  • 90425a1 Adding a query and handler
  • cbe0d92 added tests for controller
  • ddb7e66 GetCohorts Query
  • ba09732 Merge branch 'CV-491-Delete-Transfer-Cohort' into CV-498-Api-Endpoint-For-MA
  • f233951 updated API Client endpoibnt to be GetCohorts() with new request type and slight change to response type
  • 692c49e added new response and endpoint for GetCohortsByEmployer

This list of changes was auto generated.

4.1.737

09 Jan 23:27
Compare
Choose a tag to compare

Changes:

  • 492c7ff Added serverFarmId parameter to the api-app-service-certificate resource deployment.
  • 76e2ebf Changed connection string parameters to securestring type.
  • 95fa59d Changes to template.json in preparation for ASE migration.

This list of changes was auto generated.

4.1.567: Merge pull request #595 from SkillsFundingAgency/CV-461-healthcheck

05 Dec 16:08
7c2c387
Compare
Choose a tag to compare

Changes:

  • 7c2c387 Merge pull request #595 from SkillsFundingAgency/CV-461-healthcheck
  • bc439f4 Merge pull request #613 from SkillsFundingAgency/CV-507-Turn-off-Events-Api
  • ed0d42f Merge pull request #612 from SkillsFundingAgency/CV-509-Rewrite-Manage-Apprentices-Query
  • 6eac4f7 Merge pull request #601 from SkillsFundingAgency/CV-491-Delete-Transfer-Cohort
  • 1c9b3dc Merge pull request #611 from SkillsFundingAgency/Cv-495-delete-cohort-redo-work
  • a5594cb Merge branch 'master' into CV-461-healthcheck
  • 15222df Dummy Api Client implementation to turn off integration
  • 199677e Minor tweak following tech review
  • 74e10db Merge branch 'master' into CV-491-Delete-Transfer-Cohort
  • 4473d26 Merge branch 'master' into Cv-495-delete-cohort-redo-work
See More
  • 4e7e4da PR changes renaming method
  • 16104cd Adds ReservationId field and fixes Provider implementation
  • bf9bce0 Merge pull request #604 from SkillsFundingAgency/CV-497-Provider-Email-Bug
  • 29505f9 Merge pull request #603 from SkillsFundingAgency/CV-447-Delete-Apprentice
  • 6e36343 Created a GetCohortWithDraftApprenticeships extension method which now always checks the cohort exists and is not fully approved
  • c694a54 Merge branch 'master' into CV-509-Rewrite-Manage-Apprentices-Query
  • 8ee0705 renamed DeleteCohort to Delete
  • 8b245fb missed test and removed whitepspace
  • c675117 Merge branch 'master' into CV-497-Provider-Email-Bug
  • 53fd452 redid the delete cohort API
  • ba661e2 Merge branch 'CV-447-Delete-Apprentice' into Cv-491-delete-cohort-redo-work
  • c684930 Merge branch 'master' into CV-447-Delete-Apprentice
  • b048e9e Merge branch 'master' into CV-491-Delete-Transfer-Cohort
  • 960f6fe Merge pull request #600 from SkillsFundingAgency/CV-386-api-to-accept-optional-TransferSenderId
  • 68fa483 Merge branch 'master' into CV-447-Delete-Apprentice
  • ff0fde2 Merge branch 'master' into CV-497-Provider-Email-Bug
  • 6509167 Merge branch 'master' into POC_Apprenticeship_Search_Redux
  • fb15a8d Splits mapper for search result out from mapper for single record
  • 2dd0ccd Adds mapping logic
  • 016e573 Merge pull request #610 from SkillsFundingAgency/CV-449-Add-MaLegalEntityId-before-deployment
  • b231719 fix build error by making SQL adhoc scripts as None build
  • 7c24727 forced to comment out manual code as the script
  • 0819a1a merged with master
  • c56fac5 Merge branch 'master' into CV-449-Add-MaLegalEntityId-before-deployment
  • 8fbf3a3 added
  • 927f961 added column MaLegalEntityId
  • 583a724 Merge pull request #609 from SkillsFundingAgency/CV-504-Adhoc-Sql-Fix-Approved-PriceHistory
  • 806f180 Adds adhoc script
  • 9a183a6 CV-447
  • d5de074 CV-447
  • 81afcd8 cv-447
  • ff716f3 Merge branch 'master' into CV-386-api-to-accept-optional-TransferSenderId
  • f8a6929 New procs and model tweaks for search page
  • 552a1e7 Fixes issue where ukprn was lost in provider email send
  • 1f1b8a3 CV-447
  • bb8c74d Implements cohort soft deletion
  • 47a01a4 Changes deletion to an update
  • 804c69b Merge branch 'master' into CV-386-api-to-accept-optional-TransferSenderId
  • 8a94622 Schema changes
  • 045f34e Merge branch 'master' into CV-447-Delete-Apprentice
  • 1f16c77 fixed unit tests
  • f64adb7 Merge branch 'master' into CV-386-api-to-accept-optional-TransferSenderId
  • fad3514 Update readme.md
  • 5bb38b8 Merge branch 'master' into CV-447-Delete-Apprentice
  • 7906a0a Merge branch 'master' into CV-386-api-to-accept-optional-TransferSenderId
  • f9a31c4 Unwraps EmployerDetails into composite parts of AccountLegalEntity (Employer) and Account (Transfer sender) objects
  • 22c15f8 added invariant check to ensure the TransferSenderId is a valid TransferConnection for this account
  • 210ab41 CV-447
  • e8c86ef Added CohortEmployerDetails to CreateCohort calls (this contains teh ALE infor and teh Transfer Sender In)
  • 9e6b462 CV-447
  • 9568b87 added transfer sender to create cohort requests
  • bda1c37 Added transfer sender rejection reset
  • c4a89c4 Adds DeleteDraftApprenticeship method to Cohort model
  • 54b55d0 Package update
  • b9259d8 uses ping endpoint when checking FAT API

This list of changes was auto generated.

4.1.509: Merge pull request #608 from SkillsFundingAgency/Events-Upgrade-Fix

26 Nov 09:06
6d08e0e
Compare
Choose a tag to compare

Changes:

  • 6d08e0e Merge pull request #608 from SkillsFundingAgency/Events-Upgrade-Fix
  • d301e8c Fix for defect where unregistered handler was invoked
  • 83e98e7 Change VMSize
  • 3fbe200 Remove network config from preprod
  • 4643757 Revert tokenization
  • da59110 Tokenize instance count
  • 9973114 Merge pull request #596 from SkillsFundingAgency/CV-47-History-v2
  • f1bec2b Tweaks following tech review
  • b98ea19 Merge branch 'master' into CV-47-History-v2
  • 790dc57 Merge pull request #594 from SkillsFundingAgency/CV-442-emit-CohortApprovalByTransferSenderRequested
See More
  • 2447076 Adds additional tests for cohort state tracking
  • 725d88a Adds tracking session unit tests
  • 40bb125 Adds missing unit tests for initial state capture in tracking
  • 8b562f2 Test fixup
  • 003b546 Chops the tracking session into an aggregate base class
  • 0833019 Update readme.md
  • 0c143fd CV-442
  • f311494 CV-442
  • 8f9ceee CV-442
  • bacd254 Merge pull request #599 from SkillsFundingAgency/CV-485-fixed-fully-approved-cohort-command-handler
  • 4f504d5 CV-442
  • 252000e Fixes issue where deletion issue resulted in no history being written
  • 67ec566 removed ref to SFA.DAS.Apprenticeshipservice as nuget is not required
  • 6e25fef removed ref to commitmentsV2.shared and referenced the AccountApi package directly
  • 5727923 reintroduced comitmentsV2.Shared reference so we can consume teh AccountsApi
  • a15a1e8 Merge branch 'master' into CV-442-emit-CohortApprovalByTransferSenderRequested
  • 15952ad Merge branch 'master' into CV-47-History-v2
  • 2d57c50 Merge pull request #597 from SkillsFundingAgency/CV-449-post450-get-agreementid
  • 82eb402 manually created API endpoint
  • b8733ab removed anonymous types in client
  • 9bf995e Merge branch 'master' into CV-449-post450-get-agreementid
  • 7e59dd8 Merge branch 'master' into CV-442-emit-CohortApprovalByTransferSenderRequested
  • 147f377 Merge branch 'CV-396-Emit_CohortApprovedByEmployer_to_ASB'
  • fc4c81e Merge branch 'master' into CV-449-post450-get-agreementid
  • ed556e6 added registration of IEmployerAgreementService and change controller signature so it can pick up the agreementFeature from the query string
  • d1aeabd Update readme.md
  • f701faf controller returns a long nullable agreementId if not found
  • 4c9ad7d Merge pull request #591 from SkillsFundingAgency/CON-829-update-v1-api-to-dual-authentication
  • b5ae04e Merge branch 'master' into CV-449-post450-get-agreementid
  • 4fe348f added client endpoint
  • a021ae3 added more tests for new controller
  • 9f1ffc9 added interface for EmployerAgreements
  • 955e503 Update readme.md
  • 74e9626 added new agreement endpoint
  • eec2a56 removed whitespace
  • c0aee04 Merge branch 'master' into CV-449-post450-get-agreementid
  • 4d99f56 moved EmployerAgreement Service out of shared package into API application layer
  • ad3851f Merge branch 'master' into CV-396-Emit_CohortApprovedByEmployer_to_ASB
  • 602c3b3 Merge branch 'master' into CV-47-History-v2
  • d661bb9 Merge pull request #590 from SkillsFundingAgency/CV-450-Do_Not_Allow_Employers_To_Approve_Cogorts_With_Missing_Mandatory_Fields
  • 183bda5 Fixes v1 unit test accidentially broken
  • 18865f4 Improves test coverage of diff service
  • b08fd74 Adds unit test to cover PartyApprovals property
  • caebfec Undoes an accidental change to v1 code
  • 2180b4f Moved some properties from apprenticeship base class into ApprovedApprenticeship, where they belong
  • fcd57d7 Adds additional calculated properties to Cohort aggregate for state tracking purposes
  • 21a48e5 Added GetLatestAgreement to EmployerAgreementService and refactored service to simplify it
  • 212d7a6 Little tidy up
  • ea0de24 Merge branch 'master' into POC-History-Memento
  • 0c6d2d6 fixed IsCompleteForEmployer so it returns false when no apprentices
  • 2641c5d Adds domain service tests for change tracking
  • 439f87d Merge branch 'master' into CV-396-Emit_CohortApprovedByEmployer_to_ASB
  • b986ca0 Merge branch 'master' into CV-442-emit-CohortApprovalByTransferSenderRequested
  • 64f13ae Merge branch 'CV-449-check-agreement-is-signed-when-approving-cohort' into CV-450-Do_Not_Allow_Employers_To_Approve_Cogorts_With_Missing_Mandatory_Fields
  • c2eea73 Merge branch 'master' into CV-449-check-agreement-is-signed-when-approving-cohort
  • 4ba9b43 set decoding to use PublicAccountLegalEntityId
  • 61c63e1 Merge pull request #592 from SkillsFundingAgency/CV-392-ApprovedCohortReturnedToProvider-publish-to-ASB
  • 62ea44c removed MaLegalEntityId and get it from the HashedLegalEntityId
  • 06864e5 Merge branch 'CV-449-check-agreement-is-signed-when-approving-cohort' into CV-450-Do_Not_Allow_Employers_To_Approve_Cogorts_With_Missing_Mandatory_Fields
  • 7271b81 Merge remote-tracking branch 'origin/CV-442-emit-CohortApprovalByTransferSenderRequested' into CV-442-emit-CohortApprovalByTransferSenderRequested
  • 03aa59b CV-442
  • f15cbe3 Adds event handler tests
  • 7372f9e fix field name to be Id
  • b514ccd removed unused refs
  • 5f0b81d Serializes objects as XML WCF Data Contracts
  • 2467935 CV-442
  • 0acc8eb Moves diff into the event handler, allowing suppression of empty diffs
  • bdeefaa Splits out diff service
  • 7e6e679 Merge branch 'CV-449-check-agreement-is-signed-when-approving-cohort' into CV-450-Do_Not_Allow_Employers_To_Approve_Cogorts_With_Missing_Mandatory_Fields
  • 3dfb5ad Adds change tracking session factory
  • 77d299c Adds tests to state service
  • 69191d0 corrections after testing
  • fec5e1b Adds correlationId
  • d94dee6 Fixes
  • 8a8d32d WIP
  • 2415ced Removed mementos
  • ff98d45 Added unit tests for CohortApprovedByEmployer
  • 491d549 added MaLegalEntity to the GetAccountLegalEntity Api Endpoint
  • cc86adb changed cohort approval to use MaLegalEntityId instead of AccountLegalEntityId
  • a8fdba7 Removed old event handling and replaced in V2
  • 34bd92c added saving M<aLegalEntityId to the DB table
  • 815eef5 added MaLegalEntityId field and manual update script
  • 2647326 fix test name
  • 1b2fbc6 removed whitespace
  • f1165cb missed file
  • 78b9b55 added unit tests for LagacyTopicMessagePublisher
  • 93bbb10 CON-829-update-client-to-support-dual-auth
  • 843d77e introduced TopicClientFactory and added tests for handler
  • ef8cd6f added basic azureMessageBus handler into our application layer
  • 045a4a7 Merge branch 'master' into CON-829-update-v1-api-to-dual-authentication
  • 2d5fb2c CON-829-undo-test
  • a917cce CON-829-updated-client-for-testing
  • 433db0b added error logging when calling agreement service
  • 81756f1 Merge branch 'master' into CV-449-check-agreement-is-signed-when-approving-cohort
  • aa3eeec...
Read more

4.1.343

22 Oct 14:52
6460fa6
Compare
Choose a tag to compare

Changes:

  • 6460fa6 Merge pull request #586 from SkillsFundingAgency/CV-381-Fix-Issue-WithParty-TransferSender
  • f4b3b86 Merge branch 'master' into CV-381-Fix-Issue-WithParty-TransferSender
  • e226f90 Merge pull request #588 from SkillsFundingAgency/EmergencyPerformanceImprovements
  • 9b7ec74 Fix to clear TransferApprovalStatus on Send or Approval following a Transfer Rejection
  • 9a12afc Revert previous change - I'm blind!
  • e1dd484 Added index on data lock status - apprenticeship id
  • cfe7009 Added index on data lock status - apprenticeshipupdateid
  • f3cf614 Fixes issue where cohort is with transfer sender but not identified as such
  • e67f1bc Merge pull request #571 from SkillsFundingAgency/CV-369-Remove_Unused_Toggle
  • f209323 Merge branch 'master' into CV-369-Remove_Unused_Toggle
See More
  • 67beafc Merge branch 'CV-369-Remove_Unused_Toggle' of https://github.com/SkillsFundingAgency/das-commitments into CV-369-Remove_Unused_Toggle
  • 0739385 minor change
  • a9d4b9a Addressed Ian's comments
  • 6112e06 actually fixed merge conflict
  • 97deba3 removed toggles
  • 89d87a0 Merge pull request #584 from SkillsFundingAgency/CV-395-CohortTransferApprovalRequestedEvent-Handler
  • 7031e33 Merge branch 'master' into CV-395-CohortTransferApprovalRequestedEvent-Handler
  • 150afb6 Updates to readme
  • ee1ecd6 Merge branch 'master' into CV-395-CohortTransferApprovalRequestedEvent-Handler
  • 8421c0d Merge branch 'master' into CV-395-CohortTransferApprovalRequestedEvent-Handler
  • 458c8c3 added IFundingCapService to DefaultRegistry and added Event created test
  • 2f47bc0 added applicationcommand tests
  • 9072c8b removed test code
  • b4d1a32 fixed some tests
  • 4695faf FundingCapService Tests
  • 2d71032 core code for adding a transfer request
  • 4a514fb calc funding band cap (incomplete)
  • 6fcb15e transfer requests rework
  • 576104e added message handler, command handler and basic new aggregate
  • 8a21c07 minor change
  • 680e73f Addressed Ian's comments
  • 4ecd196 actually fixed merge conflict
  • d3ebc7a merged master
  • 66d9d9f removed toggles

This list of changes was auto generated.