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

feat(ios, android): add new API's, fix existing API's #261

Open
wants to merge 51 commits into
base: master
Choose a base branch
from

Conversation

hansemannn
Copy link
Contributor

@hansemannn hansemannn commented Mar 11, 2019

All changes are in parity with https://github.com/hansemannn/titanium-googlemaps.

Android:

  • Add poiclick event
  • Add containsCoordinate method
  • Add geometryContainsLocation method
  • Add geometryDistanceBetweenPoints method
  • Support encoded polylines in Polyline.points
  • Fix showAnnotations to handle multiple annotations at once
  • Fix padding property
  • Fix crash in overlay click events

iOS:

  • Refactor source (use TitaniumKit imports, remove Xcode 9 guards)
  • Add containsCoordinate method
  • Fix padding property

@hansemannn hansemannn changed the title feat(android): add „poiclick“ event feat(ios, android): add „poiclick“ event, refactor iOS Aug 31, 2019
@build
Copy link

build commented Aug 31, 2019

Fails
🚫 version bump was Major in ios/manifest but None in package.json
🚫

npm run lint failed, please check messages below for output.

Warnings
⚠️ SDK version declared in Jenkinsfile (9.0.0.v20200207060625) does not match iOS' titanium.xcconfig value (9.0.2.GA)
⚠️

🔍 Can't find junit reports at ./TESTS-*.xml, skipping generating JUnit Report.

Messages
📖

💣 Here's the output of npm run lint:

> @titanium-sdk/[email protected] lint
> npm run lint:android && npm run lint:ios && npm run lint:js && npm run lint:docs


> @titanium-sdk/[email protected] lint:android
> clang-format-lint android/src/**/*.java

Error: Formatting incorrect on "android/src/ti/map/MapModule.java", proposed changes: <?xml version='1.0'?>
<replacements xml:space='preserve' incomplete_format='false'>
<replacement offset='7134' length='1'>&#10;								   </replacement>
<replacement offset='7558' length='0'>&#10;									 </replacement>
<replacement offset='7681' length='0'> </replacement>
<replacement offset='7813' length='0'> </replacement>
<replacement offset='7904' length='1'>&#10;								   </replacement>
<replacement offset='8033' length='4'>&#10;&#10;	</replacement>
<replacement offset='8395' length='1'>&#10;									  </replacement>
<replacement offset='8553' length='1'>&#10;									  </replacement>
<replacement offset='8761' length='1'>&#10;								 </replacement>
<replacement offset='8771' length='8'>&#10;&#10;		</replacement>
</replacements>

Error: Formatting incorrect on "android/src/ti/map/PolygonProxy.java", proposed changes: <?xml version='1.0'?>
<replacements xml:space='preserve' incomplete_format='false'>
<replacement offset='4980' length='3'>&#10;			</replacement>
<replacement offset='5040' length='2'> </replacement>
</replacements>

Error: Formatting incorrect on "android/src/ti/map/PolylineProxy.java", proposed changes: <?xml version='1.0'?>
<replacements xml:space='preserve' incomplete_format='false'>
<replacement offset='1135' length='0'>&#10;</replacement>
<replacement offset='1196' length='5'>&#10;	  </replacement>
<replacement offset='1288' length='12'>&#10;							</replacement>
<replacement offset='1375' length='12'>&#10;							</replacement>
<replacement offset='1457' length='12'>&#10;							</replacement>
<replacement offset='3182' length='6'>&#10;&#10;		</replacement>
<replacement offset='6898' length='3'>&#10;			</replacement>
<replacement offset='6958' length='2'> </replacement>
<replacement offset='8928' length='6'>&#10;												</replacement>
<replacement offset='9115' length='6'>&#10;												</replacement>
</replacements>

Error: Formatting incorrect on "android/src/ti/map/TiUIMapView.java", proposed changes: <?xml version='1.0'?>
<replacements xml:space='preserve' incomplete_format='false'>
<replacement offset='2997' length='1'>&#10;			   </replacement>
<replacement offset='3050' length='16'> </replacement>
<replacement offset='7420' length='6'>&#10;&#10;		</replacement>
<replacement offset='7930' length='6'>&#10;&#10;		</replacement>
<replacement offset='8039' length='6'>&#10;&#10;		</replacement>
<replacement offset='15521' length='3'>&#10;	</replacement>
<replacement offset='24420' length='1'></replacement>
<replacement offset='24451' length='1'></replacement>
<replacement offset='24457' length='4'>&#10;&#10;	</replacement>
<replacement offset='24525' length='1'>&#10;	</replacement>
<replacement offset='25929' length='1'></replacement>
<replacement offset='25960' length='1'></replacement>
<replacement offset='28362' length='1'>&#10;	</replacement>
<replacement offset='28385' length='5'>&#10;			</replacement>
<replacement offset='28407' length='6'>&#10;&#10;		</replacement>
<replacement offset='28518' length='1'>&#10;			</replacement>
<replacement offset='28627' length='6'>&#10;&#10;		</replacement>
<replacement offset='42386' length='1'>&#10;	</replacement>
<replacement offset='42426' length='6'>&#10;&#10;		</replacement>
<replacement offset='42686' length='6'>&#10;&#10;		</replacement>
<replacement offset='42868' length='6'>&#10;&#10;		</replacement>
</replacements>

Error: Formatting incorrect on "android/src/ti/map/ViewProxy.java", proposed changes: <?xml version='1.0'?>
<replacements xml:space='preserve' incomplete_format='false'>
<replacement offset='1877' length='1'>&#10;							</replacement>
<replacement offset='1910' length='8'> </replacement>
<replacement offset='10200' length='0'> </replacement>
<replacement offset='10241' length='9'>&#10;&#10;			</replacement>
<replacement offset='13003' length='3'>&#10;	</replacement>
<replacement offset='13279' length='3'>&#10;	</replacement>
<replacement offset='27139' length='4'>&#10;&#10;	</replacement>
<replacement offset='27503' length='1'>&#10;							   </replacement>
<replacement offset='27638' length='1'>&#10;							   </replacement>
<replacement offset='27824' length='0'> </replacement>
<replacement offset='32148' length='4'>&#10;		</replacement>
<replacement offset='32199' length='4'>&#10;&#10;	</replacement>
<replacement offset='32265' length='1'>&#10;	</replacement>
<replacement offset='32267' length='4'>&#10;		</replacement>
<replacement offset='32298' length='4'>&#10;		</replacement>
<replacement offset='32338' length='3'>&#10;			</replacement>
<replacement offset='32362' length='0'> </replacement>
<replacement offset='32399' length='4'>&#10;		</replacement>
<replacement offset='32404' length='4'>&#10;		</replacement>
</replacements>

Generated by 🚫 dangerJS against 614927d

@hansemannn hansemannn changed the title feat(ios, android): add „poiclick“ event, refactor iOS feat(ios, android): add new API's, fix existing API's Nov 2, 2019
m1ga and others added 9 commits February 23, 2022 16:21
# Conflicts:
#	android/build.gradle
#	android/manifest
#	android/src/ti/map/MapModule.java
# Conflicts:
#	android/src/ti/map/MapModule.java
Fixes: Uncaught Error: Could not copy bitmap to parcel blob.
* feat: use fragment container and add raw map mode

* feat(android): implement liteMode

Fixes MOD-2587

* fix: enable lite mode

* chore: remove unused code

* fix: remove unused file

---------

Co-authored-by: Chris Williams <[email protected]>
Co-authored-by: Hans Knöchel <[email protected]>
@m1ga
Copy link
Contributor

m1ga commented May 31, 2023

@janvennemann you can run npm run lint to make dangerJS happy 😄

* feat(android): add “customIcon” property

* chore: guard label
apidoc/View.yml Outdated Show resolved Hide resolved
janvennemann and others added 2 commits June 2, 2023 18:18
* fix: only ignore tilt and bearing in lite mode

* fix: zoom out if markers are out of view

* chore: remove debug log
@hansemannn
Copy link
Contributor Author

@m1ga Thx! Unfortunately, the linting does not seem to work:

> @titanium-sdk/[email protected] lint:android
> clang-format-lint android/src/**/*.java

TypeError: Cannot read properties of undefined (reading 'stdout')

@m1ga
Copy link
Contributor

m1ga commented Jun 2, 2023

@hansemannn sorry, it's npm run format to fix the linting. npm run lint is just to check it again

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.

4 participants