Skip to content

Conversation

lizlooney
Copy link
Collaborator

Store version number in project.info.json

…le".

Replaced listModulePaths with listFilePaths.
Replaced fetchModuleContentText with fetchFileContentText.
Replaced saveModule with saveFile.
Replaced deleteModule with deleteFile.
Added comments.
Updated regular expressions so they work for modules and non-module files.
Added functions makeFilePathRegexPattern, makeFilePath, makeProjectInfoPath, isValidModuleFileName, isValidProjectInfoFileName.
Added CURRENT_VERSION.
Added type ProjectInfo.
Added functions saveProjectInfo, and deleteProjectInfo.
Call saveProjectInfo after saving or deleting a module.
Call deleteProjectInfo when deleting a project.
Updated downloadProject and uploadProject to download/upload all files in the project.
Updated processUploadedBlob to handle the project.info.json file and module files.

In module_content.ts:
Updated parseModuleContentText to check module content.
In project.ts:
Added functions fetchProjectInfo and updateProjectIfNecessary.
Call updateProjectIfNecessary in fetchProject.
@lizlooney lizlooney requested a review from alan412 August 28, 2025 06:40
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.

I wonder if we should break out the upgrade into its own file. While it does fairly little now, I can expect this will grow quite a bit in the future.

@lizlooney
Copy link
Collaborator Author

I wonder if we should break out the upgrade into its own file. While it does fairly little now, I can expect this will grow quite a bit in the future.

I've added a TODO to move it.

@lizlooney lizlooney merged commit f1dc222 into wpilibsuite:main Aug 28, 2025
1 check passed
@lizlooney lizlooney deleted the pr_project_info_json branch August 28, 2025 20:34
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