Skip to content

Conversation

@lizlooney
Copy link
Collaborator

In Storage interface:
Replaced listModules with listModulePaths.
Removed fetchModuleDateModifiedMillis

In projects.ts:
Replace listProjects with listProjectNames.
Added fetchProject.
Changed renameProject, copyProject, deleteProject, and downloadProject functions to take projectName string instead of Project.

In Module type:
Removed dateModifiedMillis, which was never used.

Updated react components Menu, ProjectManageModal, and ProjectNameComponent.

Replaced listModules with listModulePaths.
Removed fetchModuleDateModifiedMillis

In projects.ts:
Replace listProjects with listProjectNames.
Added fetchProject.
Changed renameProject, copyProject, deleteProject, and downloadProject functions to take projectName string instead of Project.

In Module type:
Removed dateModifiedMillis, which was never used.

Updated react components Menu, ProjectManageModal, and ProjectNameComponent.
@lizlooney lizlooney requested a review from alan412 August 27, 2025 04:39
Copy link
Collaborator

@alan412 alan412 left a comment

Choose a reason for hiding this comment

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

LGTM

@alan412 alan412 merged commit 998f4fe into wpilibsuite:main Aug 27, 2025
1 check passed
@lizlooney lizlooney deleted the pr_better_storage_interface branch August 28, 2025 01:05
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