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

ViewAnnotationOptions does not accept Point under Geometry #1510

Closed
johncadengo opened this issue Aug 3, 2022 · 2 comments
Closed

ViewAnnotationOptions does not accept Point under Geometry #1510

johncadengo opened this issue Aug 3, 2022 · 2 comments
Labels
bug 🪲 Something is broken!

Comments

@johncadengo
Copy link

I'm following the example and the docs.

Here's a snippet of code from the docs:

private func addViewAnnotation(at coordinate: CLLocationCoordinate2D) {
    let options = ViewAnnotationOptions(
        geometry: Point(coordinate),
        width: 100,
        height: 40,
        allowOverlap: false,
        anchor: .center
    )
    let sampleView = createSampleView(withText: "Hello world!")
    try? mapView.viewAnnotations.add(sampleView, options: options)
}

Whenever I try to add a Point, it gives me this error:

Argument type 'Point' does not conform to expected type 'GeometryConvertible'

How can I get this to work?

@johncadengo johncadengo added the bug 🪲 Something is broken! label Aug 3, 2022
@johncadengo johncadengo changed the title ViewAnnotationOptions ViewAnnotationOptions does not accept Point under Geometry Aug 3, 2022
@johncadengo
Copy link
Author

I'm on version 10.6.1

@johncadengo
Copy link
Author

Ok, I had a name conflict on Point

mapbox-github-ci-writer-public-1 bot pushed a commit that referenced this issue May 3, 2023
* Remove binary dependency on MME

* Add changelog
mapbox-github-ci-writer-public-1 bot pushed a commit that referenced this issue May 3, 2023
* Update version numbering (#1499)

* Remove binary dependency on MME (#1510)

* Remove binary dependency on MME

* Add changelog

* Mai/prepare v10.13 (#1511)

* Bump CoreMaps to v10.13.0 and Common to v23.5.0

* Update CHANGELOG.md

* Remove carplay entry from changelog

---------

Co-authored-by: Patrick Leonard <pjleonard37@users.noreply.github.com>
Co-authored-by: Roman Gardukevich <roman.gardukevich@mapbox.com>
OdNairy added a commit that referenced this issue Aug 22, 2023
* Remove binary dependency on MME

* Add changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🪲 Something is broken!
Projects
None yet
Development

No branches or pull requests

1 participant