Skip to content
New issue

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

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

Already on GitHub? # to your account

[CQ] migrate from slated-for-removal ActionEvent APIs #8018

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pq
Copy link
Contributor

@pq pq commented Apr 8, 2025

AnActionEvent.createFromDataContext is slated for removal.

EDIT: Hrmmm. Not sure how we're supposed to migrate if ActionUiKind is unavailable.

(Will have to research.)

See: #7718.

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read the Flutter Style Guide recently, and have followed its advice.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@pq pq requested a review from jwren April 8, 2025 19:58
@pq pq force-pushed the cq_actionEventAPI_migration branch from ba4b4fc to ed8587f Compare April 8, 2025 20:10
@jwren
Copy link
Member

jwren commented Apr 10, 2025

Related to #7718

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants