Repository navigation
cleanup: remove yew-ui - #788
Conversation
|
/build-and-deploy |
|
🚀 Building images and deploying preview... Started by @testradav Phase 1: Building Docker images (~10-15 min)
Phase 2: Deploying preview (~3-5 min)
⏱️ Total time expected: ~15-20 minutes I'll post updates in this thread as each phase completes. |
|
❌ Build failed Image builds failed — deployment has been skipped. Check the workflow logs for details. |
these failures are expected, since this PR itself removes yew |
When I was training deploy of online version of videocall, we discussed with Jay some repositories in /helm. Therefore, I think repositories helm/rustlemania-ui, helm/global/us-east/rustlemania-ui and helm/global/us-east/rustlemania-ui-staging can be deleted and in file deploy.sh should be done some updates: ` Should be replaced with I think these changes should fix problems with deployment and also code, which is used only for yew-ui, will be deleted. |
great catch. makes sense |
…arts to dioxus-ui
… yew URL patterns
|
/build-and-deploy |
|
🚀 Building images and deploying preview... Started by @testradav Phase 1: Building Docker images (~10-15 min)
Phase 2: Deploying preview (~3-5 min)
⏱️ Total time expected: ~15-20 minutes I'll post updates in this thread as each phase completes. |
|
❌ Build failed Image builds failed — deployment has been skipped. Check the workflow logs for details. |
|
doc updated |
Summary / Review
Final ReportVerdict:Clean. Dioxus-ui is the sole frontend across all deployment, CI/CD, testing, and documentation surfaces. No dual-UI patterns remain. |
|
/build-and-deploy |
|
🚀 Building images and deploying preview... Started by @testradav Phase 1: Building Docker images (~10-15 min)
Phase 2: Deploying preview (~3-5 min)
⏱️ Total time expected: ~15-20 minutes I'll post updates in this thread as each phase completes. |
|
❌ Build failed Image builds failed — deployment has been skipped. Check the workflow logs for details. |
this will fail until the PR is merged since the workflow is changed in this PR |
There was a problem hiding this comment.
According to what I see in dioxus-UI description later in this file, https://pr${SLOT}-dx.sandbox.videocall.rs is part for dioxus, but it removed and saved part for yew-ui.
There was a problem hiding this comment.
one of the steps was to get rid of the -dx suffix
7f228c0
into
security-union:yew-ui-removal
* cleanup: remove yew-ui (#788) * disable yew-ui * removed yew-ui * Removed yew-ui entirely and migrated all builds, deploys, and helm charts to dioxus-ui * Remove "-dx" suffix from all dioxus deployment URLs to match original yew URL patterns * tweak: Fix GHCR cleanup workflow to purge dioxus-ui PR images instead of defunct yew web-ui * update docs to reference dioxus and remove yew references --------- Co-authored-by: Antonio Estrada <antonio.estrada@hcl-software.com> * fmt changes * save --------- Co-authored-by: Antonio Estrada <antonio.estrada@hcl.com> Co-authored-by: Antonio Estrada <antonio.estrada@hcl-software.com>
* disable yew-ui * removed yew-ui * Removed yew-ui entirely and migrated all builds, deploys, and helm charts to dioxus-ui * Remove "-dx" suffix from all dioxus deployment URLs to match original yew URL patterns * tweak: Fix GHCR cleanup workflow to purge dioxus-ui PR images instead of defunct yew web-ui * update docs to reference dioxus and remove yew references * cleanup: remove yew-ui (#788) (#789) * cleanup: remove yew-ui (#788) * disable yew-ui * removed yew-ui * Removed yew-ui entirely and migrated all builds, deploys, and helm charts to dioxus-ui * Remove "-dx" suffix from all dioxus deployment URLs to match original yew URL patterns * tweak: Fix GHCR cleanup workflow to purge dioxus-ui PR images instead of defunct yew web-ui * update docs to reference dioxus and remove yew references --------- Co-authored-by: Antonio Estrada <antonio.estrada@hcl-software.com> * fmt changes * save --------- Co-authored-by: Antonio Estrada <antonio.estrada@hcl.com> Co-authored-by: Antonio Estrada <antonio.estrada@hcl-software.com> * remove mentions of sole ui * pass args to docker build for dioxus --------- Co-authored-by: Antonio Estrada <antonio.estrada@hcl-software.com> Co-authored-by: Dario A Lencina-Talarico <darioalessandrolencina@gmail.com>
…api-nats-url fix(deploy): add missing NATS_URL to meeting-api

Remove Yew-UI
Although this PR touches +100 files, it ultimately simplifies the entire application because it removes the yew-ui + dioxus-ui coexistance. Removes thousands of lines of code. This should ultimately make the application easier to manage and digest.
76 files deleted, 18,666 lines of code removed
Workspace & Build
Docker/Compose/Helm
CI/CD (6 workflows)
E2E & Docs
Removed
Type of Change
Testing
Other