Skip to content

Increase demo cycle time from 15s to 5mins#6549

Draft
orangesurf wants to merge 2 commits into
masterfrom
orangesurf/conf-disp
Draft

Increase demo cycle time from 15s to 5mins#6549
orangesurf wants to merge 2 commits into
masterfrom
orangesurf/conf-disp

Conversation

@orangesurf

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings June 9, 2026 09:31
@cla-bot cla-bot Bot added the cla-signed label Jun 9, 2026
Comment thread frontend/src/app/shared/common.utils.ts Fixed
Comment thread frontend/src/app/shared/common.utils.ts Fixed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adjusts the demo auto-redirect cadence in the Angular frontend so the demo “cycle” stays on each page longer before rotating.

Changes:

  • Increased the demo redirect delay from 15 seconds to 5 minutes in handleDemoRedirect.

Comment thread frontend/src/app/shared/common.utils.ts Outdated
Comment on lines +229 to +233
@@ -230,7 +230,7 @@ export function handleDemoRedirect(route: ActivatedRoute, router: Router) {
const index = path.indexOf(params.next);
if (index >= 0) {
const nextPath = path[(index + 1) % path.length];
setTimeout(() => { window.location.replace(`${params.next}?next=${nextPath}`); }, 15000);
setTimeout(() => { window.location.replace(`${params.next}?next=${nextPath}`); }, 300000);
@orangesurf orangesurf marked this pull request as draft June 9, 2026 09:53
@nymkappa nymkappa requested a review from Copilot June 9, 2026 10:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment on lines +226 to +237
let demoRedirectTimer: ReturnType<typeof setTimeout>;
route.queryParams
.subscribe(params => {
if (params.next) {
const path = ['/', '/acceleration', '/mining', '/lightning'];
const index = path.indexOf(params.next);
if (index >= 0) {
const nextPath = path[(index + 1) % path.length];
setTimeout(() => { window.location.replace(`${params.next}?next=${nextPath}`); }, 15000);
clearTimeout(demoRedirectTimer);
demoRedirectTimer = setTimeout(() => {
window.location.replace(`${path[index]}?next=${nextPath}`);
}, 300000);
Comment on lines 227 to +237
route.queryParams
.subscribe(params => {
if (params.next) {
const path = ['/', '/acceleration', '/mining', '/lightning'];
const index = path.indexOf(params.next);
if (index >= 0) {
const nextPath = path[(index + 1) % path.length];
setTimeout(() => { window.location.replace(`${params.next}?next=${nextPath}`); }, 15000);
clearTimeout(demoRedirectTimer);
demoRedirectTimer = setTimeout(() => {
window.location.replace(`${path[index]}?next=${nextPath}`);
}, 300000);
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants