Skip to content

Comments

fix(web): drop components no longer needed#1901

Merged
dgdavid merged 2 commits intostorage-config-uifrom
drop-old-storage-code
Jan 16, 2025
Merged

fix(web): drop components no longer needed#1901
dgdavid merged 2 commits intostorage-config-uifrom
drop-old-storage-code

Conversation

@dgdavid
Copy link
Contributor

@dgdavid dgdavid commented Jan 15, 2025

The rewrite of the storage interface obsoletes a lot of components in storage and core namespaces. Let's drop them by now and just rescue needed parts, if any, when proceed.

As a side-effect, it reduces the list of skipped unit test from 54 to 16.

Since they does not met requirements of the new storage interface.
Which are no longer referenced and probably no longer needed.
@dgdavid dgdavid requested review from imobachgs and joseivanlopez and removed request for imobachgs January 15, 2025 18:39
Copy link
Contributor

@imobachgs imobachgs left a comment

Choose a reason for hiding this comment

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

+0/-5.132... awesome! 😎

@dgdavid dgdavid merged commit 544f0cd into storage-config-ui Jan 16, 2025
2 checks passed
@dgdavid dgdavid deleted the drop-old-storage-code branch January 16, 2025 11:42
dgdavid added a commit that referenced this pull request Jan 16, 2025
Simply deletes a test file overlooked at #1901
@imobachgs imobachgs mentioned this pull request Feb 26, 2025
imobachgs added a commit that referenced this pull request Feb 26, 2025
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