Skip to content

Conversation

@Phil-NHS
Copy link
Contributor

@Phil-NHS Phil-NHS commented Aug 14, 2025

This PR is not the PR into LH Kentia. It is one of the steps in the process. The purpose of the PR is to share the steps if desired.

JIRA link

TD-5924

AnjuJose011 and others added 30 commits July 3, 2025 08:33
…ributeScreen-AccessDenied

 TD-5727-Fix for Full user who is not a catalogue editor/admin can access My contribution page by navigating to the link directly
…ursetray-Reorder-resource-tabs

TD-5700-Reorder resource tabs on LH home page
…s/TD-5856_reload_moodle_user

Develop/fixes/td 5856 reload moodle user
…ure/TD-5888-Make-LH-able-to-Host-Blazor

Develop/feature/td 5888 make lh able to host blazor

Merging so against one task branch which we will take only certain files from anyway
…ompatible and structure, interfaces are duplicated
…sed for cicd following text replacement for tokens
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Felt could leave this in as is blazor side

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gitincludes are just because without the search stuff git wont track the structure so it needs a place holder file

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is so we can trim the file name and use it in cicd plus a name replace. This approach is not the only approach and i am not familiar with with the pipelines

@Phil-NHS
Copy link
Contributor Author

@AnjuJose011 @ArunimaGeorge This is an interim PR so i haven't put you as reviewers. Just tagging you in case you had wanted to give it a look.

@Phil-NHS Phil-NHS merged commit 87f3f5c into Develop/Feature/TD-5930-Blazor-Implementation-Integration-LH Aug 21, 2025
2 of 3 checks passed
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.

3 participants