Skip to content

Conversation

@neelmistry94
Copy link
Contributor

@neelmistry94 neelmistry94 commented Jun 2, 2021

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

Fixes: An issue where location puck was not tracking course

Pull request checklist:

  • Briefly describe the changes in this PR.
  • Apply changelog label ('breaking change', 'bug 🪲', 'build', 'docs', 'feature 🍏', 'performance ⚡', 'testing 💯') or use the label 'skip changelog'

Summary of changes

This change brings the support for course when updating the locationPuck. There is a new option that the developer can specify whether the bearing should be updated via heading: CLHeading or via course: CLLocationDirection

@neelmistry94 neelmistry94 added the bug 🪲 Something is broken! label Jun 2, 2021
@neelmistry94 neelmistry94 force-pushed the nm/location_support_course branch from b27f061 to 7ef4b67 Compare June 3, 2021 19:24
adding back correct puck image

support course via options

merge conflict

update puckBearingSource so that it is a stored property of Puck

fix broken tests

test that puck bearing source propogates changes from manager to puck

clearing up docs and formatting

remove unneeded function

build fix

prevent premature creation of puck
@neelmistry94 neelmistry94 force-pushed the nm/location_support_course branch from 12d4135 to 68a138d Compare June 7, 2021 13:05
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

Successfully merging this pull request may close these issues.

3 participants