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

Added Pins (MKPointAnnotations) to map #867

Closed

Conversation

DanielMSchmidt
Copy link
Contributor

As I couldn't find more documentation / tests for the annotations, I assumed that there are no points where I should have added documentation / tests. Closes #276

Daniel Schmidt and others added 9 commits March 29, 2015 22:05
* master: (32 commits)
  Update React.podspec
  Move libicucore to RCTWebSocketDebugger subspec
  Update React.podspec
  fix website
  Fix "next" link on NativeModules
  Add a comment about Live Reload
  Add screenshots to make it clearer
  Fix typo
  [ReactNative][Docs] Document how to link libraries
  Added #content hash to debugging link so that it scrolls to start of text on mobile
  Added tip to configure 'Pause On Caught Exceptions' flag
  Add basic documentation for debugging react native apps
  Add f8 video, changelog and jsjabber
  0.3.1
  Bring back accidentally reverted changes from 89fc8a4
  Autobuild website on Travis
  Revert searches to SearchScreen.js - merging internally
  Fix typo and add comments to the example app
  Update tutorial - use sample data
  Remove blocked rotten tomatoes api key
  ...
…e/map-with-pins

* 'master' of github.com:facebook/react-native:
  Fix Objective-C example in README
  Add '#content' to link
  Link to troubleshooting from getting started page
  Fix docs links
  wrong javascript in example
  Update NativeModulesIOS.md
  Add import RCTBridge.h for event sending example
  Fix broken link
  Fixed typo in tutorial
  (fix typo) Debugging.md
  Copy fix: Asyncronously -> Asynchronously
  Update GettingStarted.md
  Add recurring issues for troubleshooting
  Add note for hardware keyboard in simulator
  corrects typo in NativeModulesIOS README
  Add possessive case to React
  Adding the 'running the examples' instructions to the end of the README.md
  Adjust the plurality of goal to goals
…e/map-with-pins

# By Christopher Chedeau (77) and others
# Via Christopher Chedeau (32) and others
* 'master' of github.com:facebook/react-native: (565 commits)
  [ReactNative] Use a single DisplayLink held by the bridge
  Replace ES6Promise with Promise
  MapView to support MKPointAnnotation using new attribute annotate in Map...
  [ReactNative] Move image asset loading to a queue
  Fix typo in NativeModulesIOS.md
  [react-packager] Add Array.prototype.es6 polyfill
  Fix typo in PanResponder documentation
  [Bridge] `RCT_REMAP_METHOD(js_name, selector)`
  [react-packager] Support @nx resolution postfix for assets
  Add flowconfig to SampleApp
  [ReactNative] allow running JS app server for Android from fbobjc
  [react-packager] Update sane to get windows fixes (and others)
  change to git clone https://github.com/facebook/react-native.git
  React Native: Sync Transforms and Enable Trailing Commas
  React Native: Add String.prototyp.es6 Polyfill
  [ReactNative] Fix logging
  React Native: Fix InteractionManager Race Condition
  [ipasize][ReactKit] Begin to ensmallen RCTConvert
  [ReactNative] Reverted D1986572
  [ReactNative] Make RCTRootView propage motions
  ...

# Conflicts:
#	React/Views/RCTMap.m
#	React/Views/RCTMapManager.m
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 15, 2015
@DanielMSchmidt
Copy link
Contributor Author

The tests seem to be aborted due to a timeout, would it be possible to run the build again?

@nicklockwood
Copy link
Contributor

How are pins different from annotations? They seem to be duplicating the same concept...

@DanielMSchmidt
Copy link
Contributor Author

I checked it, they are the same, I was simply too slow implementing this. Thanks for your help

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Possibility to show pins on MapView (MKPointAnnotation)
4 participants