Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

set the page after initializing the Handler #1813

Merged
merged 1 commit into from
Aug 25, 2023
Merged

Conversation

dansiegel
Copy link
Contributor

GitHub Issue (If applicable): #

  • n/a

PR Type

What kind of change does this PR introduce?

  • Bugfix

What is the current behavior?

We set the page in the window constructor... this can generate an exception where the Window Handler is not yet initialized.

What is the new behavior?

Set the Page on the Window after the Window Handler is set.

@nickrandolph nickrandolph merged commit a7151a1 into main Aug 25, 2023
12 of 13 checks passed
@nickrandolph nickrandolph deleted the dev/ds/maui-bug branch August 25, 2023 18:12
@nickrandolph
Copy link
Contributor

@mergify backport legacy/2x

@mergify
Copy link

mergify bot commented Aug 30, 2023

backport legacy/2x

✅ Backports have been created

@nickrandolph
Copy link
Contributor

@mergify backport release/stable/2.5

@mergify
Copy link

mergify bot commented Aug 30, 2023

backport release/stable/2.5

✅ Backports have been created

nickrandolph added a commit that referenced this pull request Aug 30, 2023
set the page after initializing the Handler (backport #1813)
nickrandolph added a commit that referenced this pull request Aug 30, 2023
….5/pr-1813

set the page after initializing the Handler (backport #1813)
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.

None yet

2 participants