Skip to content

Conversation

@evil159
Copy link
Contributor

@evil159 evil159 commented Nov 11, 2025

What does this pull request do?

Fixes annotation interaction handlers not being passed to platform SDK upon annotation update.

What is the motivation and context behind this change?

#1026 https://mapbox.atlassian.net/browse/MAPSFLT-413

Pull request checklist:

  • Add a changelog entry.
  • Write tests for all new functionality. If tests were not written, please explain why.
  • Add documentation comments for any added or updated public APIs.

PRs must be submitted under the terms of our Contributor License Agreement CLA.

Copy link
Contributor

@maios maios left a comment

Choose a reason for hiding this comment

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

LGTM! Can we add some tests for this use case?

@evil159
Copy link
Contributor Author

evil159 commented Nov 11, 2025

LGTM! Can we add some tests for this use case?

To test this we'd need to simulate touch events from the native side, not sure how to do this from the Flutter side.

pjleonard37
pjleonard37 previously approved these changes Nov 11, 2025
Co-authored-by: Patrick Leonard <pjleonard37@users.noreply.github.com>
pjleonard37
pjleonard37 previously approved these changes Nov 11, 2025
Copy link
Contributor

@pjleonard37 pjleonard37 left a comment

Choose a reason for hiding this comment

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

👍

@evil159 evil159 merged commit 0c0f179 into main Nov 11, 2025
7 checks passed
@evil159 evil159 deleted the rl/fix_annotation_interactions branch November 11, 2025 13:58
evil159 added a commit that referenced this pull request Nov 11, 2025
* fix annotation interactions stop working after annotation update

* add changelog

* Update CHANGELOG.md

Co-authored-by: Patrick Leonard <pjleonard37@users.noreply.github.com>

---------

Co-authored-by: Patrick Leonard <pjleonard37@users.noreply.github.com>
@evil159 evil159 mentioned this pull request Nov 11, 2025
3 tasks
evil159 added a commit that referenced this pull request Nov 11, 2025
* Annotation improvements (#1064)

* allAnnotations + customData

* add changelog entry

* fix annotation interactions stop working after annotation update (#1063)

* fix annotation interactions stop working after annotation update

* add changelog

* Update CHANGELOG.md

Co-authored-by: Patrick Leonard <pjleonard37@users.noreply.github.com>

---------

Co-authored-by: Patrick Leonard <pjleonard37@users.noreply.github.com>

---------

Co-authored-by: Patrick Leonard <pjleonard37@users.noreply.github.com>
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.

3 participants