Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

Correct installer page routes and conditions #508

Merged
merged 2 commits into from
Nov 16, 2021

Conversation

jpnurmi
Copy link
Contributor

@jpnurmi jpnurmi commented Nov 16, 2021

No description provided.

@jpnurmi
Copy link
Contributor Author

jpnurmi commented Nov 16, 2021

A clear and easy to read flow diagram adapted from https://miro.com/app/board/o9J_klzAnOw=/ 😄

image

@jpnurmi
Copy link
Contributor Author

jpnurmi commented Nov 16, 2021

P.S. There will be an integration test for navigating through all the possible routes: #283

Copy link
Contributor

@oSoMoN oSoMoN left a comment

Choose a reason for hiding this comment

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

Just a minor style nit, otherwise LGTM, feel free to merge.

packages/ubuntu_desktop_installer/lib/installer.dart Outdated Show resolved Hide resolved
@oSoMoN oSoMoN merged commit f4fe1a2 into canonical:main Nov 16, 2021
@jpnurmi jpnurmi deleted the installer-routes branch November 16, 2021 12:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants