Skip to content
This repository has been archived by the owner on Sep 13, 2024. It is now read-only.

chore: Remove unused _*.json metadata #151

Merged
merged 1 commit into from
Jul 26, 2024
Merged

Conversation

seakayone
Copy link
Contributor

@seakayone seakayone commented Jul 26, 2024

Those with a _ are not visible in meta.dasch.swiss. There is code which ignores these files in the service.

I think we should remove them or at least fix these files, e.g. one of the files contains a duplicate shortcode.

@seakayone seakayone requested a review from danielasubotic July 26, 2024 05:28
@mpro7 mpro7 self-requested a review July 26, 2024 06:34
Copy link
Collaborator

@mpro7 mpro7 left a comment

Choose a reason for hiding this comment

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

I would keep these files in the old repo.

@seakayone
Copy link
Contributor Author

I would keep these files in the old repo.

Given @danielasubotic approval and the benefits of reducing clutter and improving efficiency, I believe we should proceed with removing these unused files, as they can always be retrieved from version control if needed.

@seakayone seakayone requested a review from mpro7 July 26, 2024 08:30
Copy link
Contributor

@danielasubotic danielasubotic left a comment

Choose a reason for hiding this comment

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

looks good to me!

@seakayone seakayone merged commit 614314d into main Jul 26, 2024
5 checks passed
@seakayone seakayone deleted the chore/remove-unused-json branch July 26, 2024 13:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants