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

chore!: remove all leftover deprecated symbols #1730

Merged
merged 6 commits into from
Nov 22, 2023

Conversation

JaffaKetchup
Copy link
Member

Requires #1726 to function correctly. Makes #1724 redundant.

Also includes a minor refactoring of example app Live Web Demo introduction dialog.

Minor refactoring of example app Live Web Demo introduction dialog
@JaffaKetchup
Copy link
Member Author

@ibrierley @mootw @TesteurManiak @josxha Would be great if this could be reviewed! It should hopefully make things simpler/less confusing for other PRs.

@josxha
Copy link
Contributor

josxha commented Nov 22, 2023

Can do!

Copy link
Contributor

@josxha josxha left a comment

Choose a reason for hiding this comment

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

Only one consideration. Other than that it looks good to me.

example/lib/pages/home.dart Outdated Show resolved Hide resolved
@JaffaKetchup JaffaKetchup requested a review from josxha November 22, 2023 19:47
@josxha
Copy link
Contributor

josxha commented Nov 22, 2023

The changes are looking good to me. 👍🏼
Windows CD builds are currently broken. Seems like the build directory has changed?

  • old: D:\a\flutter_map\flutter_map\example\build\windows\runner\Release\example.exe
  • new: D:\a\flutter_map\flutter_map\example\build\windows\x64\runner\Release\example.exe

@josxha
Copy link
Contributor

josxha commented Nov 22, 2023

Ready to merge 🎉

@JaffaKetchup
Copy link
Member Author

JaffaKetchup commented Nov 22, 2023

Ah, already fixed Inno in 706ce4, but shouldn't cause conflicts so no worries.

@JaffaKetchup JaffaKetchup merged commit 832440b into master Nov 22, 2023
7 checks passed
@JaffaKetchup JaffaKetchup deleted the remove-deprecations branch November 22, 2023 20:29
josxha added a commit that referenced this pull request Nov 29, 2023
@josxha josxha added this to the v7.0 milestone Dec 2, 2023
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.

[BUG] InteractionFlag.enableScrollWheel has no effect
2 participants