Now that #12136 is merged, we should fix the following issues: - [x] Migrate `AddViteApp` and `WithNpmPackageInstallation` from https://github.com/CommunityToolkit/Aspire/blob/main/src/CommunityToolkit.Aspire.Hosting.NodeJS.Extensions/NodeJSHostingExtensions.cs to the dotnet/aspire repo. (#12283) - This way our template isn't dependent on an external repo, which has a dependency on us. The circular dependency isn't ideal. - cc @aaronpowell - [x] Remove the dockerfile and replace with a mechanism that publishes the frontend static files to the built-in python dockerfile #12162 - [x] Should we rename the `api_service` folder? How about `app`? (#12215) - [x] The python app should `.WaitFor` the `cache` (#12215) - [x] Make `redis` cache optional (#12215) - [x] The python app returns `500` during `aspire run` because the `static` folder doesn't exist. (#12215) cc @davidfowl
Now that #12136 is merged, we should fix the following issues:
AddViteAppandWithNpmPackageInstallationfrom https://github.com/CommunityToolkit/Aspire/blob/main/src/CommunityToolkit.Aspire.Hosting.NodeJS.Extensions/NodeJSHostingExtensions.cs to the dotnet/aspire repo. (Add ViteApp and Npm Package Manager #12283)api_servicefolder? How aboutapp? (Enhance python starter template #12215).WaitForthecache(Enhance python starter template #12215)rediscache optional (Enhance python starter template #12215)500duringaspire runbecause thestaticfolder doesn't exist. (Enhance python starter template #12215)cc @davidfowl