Skip to content

Deleting unused templated which are related to kf#465

Merged
Azher2Ali merged 1 commit into
developfrom
chore/remove_unusedtemplates
Aug 8, 2024
Merged

Deleting unused templated which are related to kf#465
Azher2Ali merged 1 commit into
developfrom
chore/remove_unusedtemplates

Conversation

@Azher2Ali
Copy link
Copy Markdown
Contributor

@Azher2Ali Azher2Ali commented Jul 31, 2024

Removal of KF-Related Templates
Summary:

The current templates in the SCORe client were originally designed to represent the format related to the Kids First (KF) project. Since the KF project is no longer in use within the SCORe client, we are removing the entire directory associated with these templates.

Details:

Directory Removed:
The folder and all its contents located at score-client/src/main/resources/templates have been completely removed from the repository.

This cleanup helps streamline the codebase and eliminate legacy components that are no longer needed. Please ensure that your workflow does not depend on these templates, as they are now deprecated. #459

@Azher2Ali Azher2Ali requested review from joneubank and leoraba August 2, 2024 14:01
Copy link
Copy Markdown
Contributor

@leoraba leoraba left a comment

Choose a reason for hiding this comment

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

LGTM, however I see these templates are still used on manifest/kf/ classes which I suspect those classes are deleted on a separate PR.

@Azher2Ali
Copy link
Copy Markdown
Contributor Author

LGTM, however I see these templates are still used on manifest/kf/ classes which I suspect those classes are deleted on a separate PR.

The templates which you have mentioned are handled in seperate PR, in which we have deleted all those templates which are related to KF or ICGC

@Azher2Ali Azher2Ali merged commit 930e561 into develop Aug 8, 2024
@Azher2Ali Azher2Ali deleted the chore/remove_unusedtemplates branch August 8, 2024 18:14
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.

2 participants