Skip to content

Fixing black page showing up sometimes when login with the new browse… - #18

Open
manikonda wants to merge 1 commit into
att:att/4.22.0.0from
manikonda:FixBlankPageShowingUpOnFirstLoginSometimes
Open

manikonda wants to merge 1 commit into
att:att/4.22.0.0from
manikonda:FixBlankPageShowingUpOnFirstLoginSometimes

Conversation

@manikonda

Copy link
Copy Markdown

Description

This PR fixes black page showing up sometimes when login with the new browser session. This ensures Vue Router re-resolves /dashboard against the route table after the dynamic routes have been added via router.addRoute(), rather than reusing the stale to object that was resolved when /dashboard didn't exist yet (causing the blank page on first login).

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)
  • Build/CI
  • Test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

How Has This Been Tested?

Login into the Cloudstack UI with the new browser (clean cache before opening) and try to login to the main landing page. After the login (SSO or MFA) it should display the default home/route page, which is dashboard. Before this change, sometime it just shows empty page due to race conditions of the dataset loads and APIs executions in parallel

…r session

Fixing black page showing up sometimes when login with the new browser session.
This ensures Vue Router re-resolves /dashboard against the route table after the dynamic routes have been added via router.addRoute(), rather than reusing the stale to object that was resolved when /dashboard didn't exist yet (causing the blank page on first login).
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.

1 participant