Skip to content

Feature/canvas android#5

Merged
sorodrigo merged 26 commits intomasterfrom
feature/canvas-android
May 29, 2017
Merged

Feature/canvas android#5
sorodrigo merged 26 commits intomasterfrom
feature/canvas-android

Conversation

@j8seangel
Copy link
Copy Markdown

@j8seangel j8seangel commented May 26, 2017

This PR includes the custom Canvas Android implementation what adds the below features, ONLY IN ANDROID, TODO: IOS

  • Offline support,
  • Custom canvas filter by dates layer.
  • New interaction layer to highlight the area selected (TODO: improve the highlight size in zoom level changes)

This component extends the UrlTile one and receives the following new params:

  • areaId: necessary to generate the cached folder.
  • alertType: supports VIIRS("viirs") and GLAD ("umd_as_it_happens") layers, also mandatory for the cached folder.
  • isConnected: gets the online status in the app to get from the server or from the cached folder
  • min and maxDate: dates with the days count from the origin in GLAD('20150101') and 1(today)-8(last-week) to VIIRS.

@j8seangel
Copy link
Copy Markdown
Author

Please, don't merge until the master branch is sync with the airbnb master branch to keep the repos synched and make the conflicts easier to solve.
Also remember to remove #feature/canvas-android from the FW repo package.json in "react-native-maps": "Vizzuality/react-native-maps#feature/canvas-android"

@j8seangel
Copy link
Copy Markdown
Author

j8seangel commented May 26, 2017

Feel free to merge! (#6)

Copy link
Copy Markdown

@sorodrigo sorodrigo left a comment

Choose a reason for hiding this comment

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

Great! I left you some marks on some logs I found.


@Override
public String getName() {
Log.d("Map", "Get name");
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

is this a console log?

const { region } = this.state;
const { region, coordinates } = this.state;
const hasCoordinates = (coordinates.tile && coordinates.tile.length === 3) || false;
console.log(coordinates);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

console log


@Override
public String getName() {
Log.d("Map", "Get name");
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

log


if (this.coordinates != null) {
int[] tile = this.coordinates.getTile();
Log.d("tile Position: ", x + "-" + y + "-" + zoom);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

log

j8seangel added 2 commits May 29, 2017 12:36
…native-maps into feature/canvas-android

* 'feature/canvas-android' of github.com:Vizzuality/react-native-maps: (24 commits)
  v0.15.2 (react-native-maps#1352)
  Remove AIRGMSPolyline.h and AIRGMSPolyline.m references from AirMaps.xcodeproj (react-native-maps#1351)
  v0.15.1 (react-native-maps#1347)
  fix compile error in rn version >= 0.40 (react-native-maps#1341)
  Add Marker rotation for Google Maps on iOS (react-native-maps#1326)
  Fixing reference to AirMapsExplorer in installation docs (react-native-maps#1328)
  Update README: Use callback in `ref` attribute (react-native-maps#1345)
  [iOS] Added onPress support for Polyline on Google Maps (react-native-maps#1194)
  [rnpm] Fix sourceDir for Android (react-native-maps#1313)
  Update license date (react-native-maps#1316)
  Fix overlay issues in Android introduced in 0.13.1 (react-native-maps#1311)
  v0.15.0 (react-native-maps#1305)
  fixing code snippet (react-native-maps#1196)
  small typo fixed (react-native-maps#1211)
  Update installation.md (react-native-maps#1226)
  Fixed path in `android/settings.gradle` (react-native-maps#1230)
  Add babelrc to npmignore (react-native-maps#1246)
  Update installation.md (react-native-maps#1179)
  Update docs to specify how to access event data (react-native-maps#1178)
  Update path in android installation (react-native-maps#1249)
  ...
@j8seangel
Copy link
Copy Markdown
Author

Logs removed

@sorodrigo sorodrigo merged commit 4e9fd7a into master May 29, 2017
@sorodrigo sorodrigo deleted the feature/canvas-android branch May 29, 2017 10:38
@geriux
Copy link
Copy Markdown

geriux commented May 29, 2017

Awesome work!!!!!!

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