Skip to content

implement oltd for withdrawn apprenticeship - #802

Open
skumardfe wants to merge 9 commits into
masterfrom
APPMAN-2724-prevent-provider-OLTD-create-request-ilr-withdrawn
Open

implement oltd for withdrawn apprenticeship#802
skumardfe wants to merge 9 commits into
masterfrom
APPMAN-2724-prevent-provider-OLTD-create-request-ilr-withdrawn

Conversation

@skumardfe

@skumardfe skumardfe commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@das-github-app

das-github-app Bot commented Jun 24, 2026

Copy link
Copy Markdown

🔒 Please remember to check any packages used by this application to ensure they are up to date @skumardfe. cc/

@das-github-app

das-github-app Bot commented Jun 24, 2026

Copy link
Copy Markdown

🔒 Please remember to check any packages used by this application to ensure they are up to date @skumardfe. cc/

@das-github-app

das-github-app Bot commented Jun 25, 2026

Copy link
Copy Markdown

🔒 Please remember to check any packages used by this application to ensure they are up to date @skumardfe. cc/

@skumardfe skumardfe changed the title implment oltd for withdrwan apprenticeship implement oltd for withdrawn apprenticeship Jun 25, 2026
@das-github-app

das-github-app Bot commented Jun 25, 2026

Copy link
Copy Markdown

🔒 Please remember to check any packages used by this application to ensure they are up to date @skumardfe. cc/

cofaulco
cofaulco requested changes Jun 26, 2026
View reviewed changes
Comment thread ...ents/SFA.DAS.ProviderCommitments.Web/Controllers/OverlappingTrainingDateRequestController.cs
Comment thread ...rCommitments/SFA.DAS.ProviderCommitments.Web/Views/Shared/_OverlapIlrWithdrawnPartial.cshtml Outdated
Comment thread ...rCommitments/SFA.DAS.ProviderCommitments.Web/Views/Shared/_OverlapIlrWithdrawnPartial.cshtml Outdated
Comment thread ...rCommitments/SFA.DAS.ProviderCommitments.Web/Views/Shared/_OverlapIlrWithdrawnPartial.cshtml Outdated
Comment thread ...ers/OverlappingTrainingDateRequestControllerTests/WhenIGetOverlapOptionsForChangeEmployer.cs
@das-github-app

das-github-app Bot commented Jun 30, 2026

Copy link
Copy Markdown

🔒 Please remember to check any packages used by this application to ensure they are up to date @skumardfe. cc/

@das-github-app

das-github-app Bot commented Jul 7, 2026

Copy link
Copy Markdown

🔒 Please remember to check any packages used by this application to ensure they are up to date @skumardfe. cc/

@das-github-app

das-github-app Bot commented Jul 10, 2026

Copy link
Copy Markdown

🔒 Please remember to check any packages used by this application to ensure they are up to date @skumardfe. cc/

cofaulco
cofaulco previously approved these changes Jul 13, 2026
View reviewed changes
@das-github-app

das-github-app Bot commented Aug 4, 2026

Copy link
Copy Markdown

🔒 Please remember to check any packages used by this application to ensure they are up to date @skumardfe. cc/

@das-github-app

das-github-app Bot commented Aug 6, 2026

Copy link
Copy Markdown

🔒 Please remember to check any packages used by this application to ensure they are up to date @skumardfe. cc/

@sonarqubecloud

sonarqubecloud Bot commented Aug 6, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cofaulco cofaulco cofaulco left review comments
@chrisfoster76 chrisfoster76 Awaiting requested review from chrisfoster76 chrisfoster76 is a code owner
@Pauljgraham Pauljgraham Awaiting requested review from Pauljgraham Pauljgraham is a code owner
@witpo witpo Awaiting requested review from witpo witpo is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@skumardfe @cofaulco @sony-kotha