Esri / developer-support

Proof of concept developer code and samples to help be successful with all ArcGIS developer products (Python, NET, JavaScript, Android…). The repository is designed to be an exchange for sharing coding conventions and wisdom to developers at all skill levels.
Apache License 2.0
266 stars 164 forks source link

runtime-android/sync-gdb-in-background missing res folder #238

Closed doneill closed 8 years ago

doneill commented 8 years ago

I am trying to update the sync-gdb-in-background android demo, beyond adding the gradle wrapper and updating the ignore list there are references to resources but the res directory is missing. I can try to piece together a res file based on what the project needs, but best if we can introduce a proper res folder.

doneill commented 8 years ago

I've resolved icons, styles, but the strings will be an issue :-)

nohe427 commented 8 years ago

Hey @doneill,

Thanks for using my sample! This is the string resources I am using:

<resources>
  <string name="app_name">BackgroundSyncGeodatabase</string>
  <string name="sync_notification_title">Portal View</string>
  <string name="sync_updating">GDB Sync in Progress</string>
  <string name="sync_done">GDB Sync Completed</string>
  <string name="sync_failed">GDB Sync Failed</string>
</resources>

I didn't include the res file so as to not bloat the repository with extra images and strings. I was worried I was sending too much data.

Are there some other items I can assist with in regards to this sample?

nohe427 commented 8 years ago

Hey @jgravois & @doneill !

The res file did not bloat the repository as much as I originally assumed it would. I went ahead and included it in pull request #239

Again, please let me know if there is anything I can assist with :space_invader:

doneill commented 8 years ago

Thanks @nohe427, Yes you should include everything to allow a developer to build the module. I updated the gitignore list which details the files that should not be included. I also added the gradle wrapper jar file and scripts, I know this repo explicitly ignores jar files but we need it such that developers do not need to have Gradle on their machine to run the demo. Google recommends this.

nohe427 commented 8 years ago

Hey no worries. If we could chat about what should be required for Android samples later, I would appreciate it On Jun 22, 2016 3:58 PM, "doneill" notifications@github.com wrote:

Thanks @nohe427 https://github.com/nohe427, Yes you should include everything to allow a developer to build the module. I updated the gitignore list which details the files that should not be included. I also added the gradle wrapper jar file and scripts, I know this repo explicitly ignores jar files but we need it such that developers do not need to have Gradle on their machine to run the demo. Google recommends this.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Esri/developer-support/issues/238#issuecomment-227858621, or mute the thread https://github.com/notifications/unsubscribe/ADEBRd2bUCsnDN6DzmvSWsZekHYZ2dWWks5qOZPIgaJpZM4I7-BA .

doneill commented 8 years ago

@nohe427 Merging that PR helped, but nothing is happening in the app. I will dig deeper into the app source but on start I get the following app screenshot:

bg-sync

nohe427 commented 8 years ago

Give it two minutes On Jun 22, 2016 4:08 PM, "doneill" notifications@github.com wrote:

@nohe427 https://github.com/nohe427 Merging that PR helped, but nothing is happening in the app. I will dig deeper into the app source but on start I get the following app screenshot:

[image: bg-sync] https://cloud.githubusercontent.com/assets/1308001/16281710/750029e4-387a-11e6-9441-2f45d6f402a1.png

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Esri/developer-support/issues/238#issuecomment-227861480, or mute the thread https://github.com/notifications/unsubscribe/ADEBRQazRtYkRwicclnE-0dLlVhU3VD9ks5qOZZYgaJpZM4I7-BA .

nohe427 commented 8 years ago

I have it set up with alarm manager in the intent service On Jun 22, 2016 4:08 PM, "doneill" notifications@github.com wrote:

@nohe427 https://github.com/nohe427 Merging that PR helped, but nothing is happening in the app. I will dig deeper into the app source but on start I get the following app screenshot:

[image: bg-sync] https://cloud.githubusercontent.com/assets/1308001/16281710/750029e4-387a-11e6-9441-2f45d6f402a1.png

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Esri/developer-support/issues/238#issuecomment-227861480, or mute the thread https://github.com/notifications/unsubscribe/ADEBRQazRtYkRwicclnE-0dLlVhU3VD9ks5qOZZYgaJpZM4I7-BA .

doneill commented 8 years ago

@nohe427 So what is the user workflow? Do you need to sync a database from another app? Trying to verify this is working as expected.

nohe427 commented 8 years ago

Hey @doneill , I think we resolved a lot of this offline. Do you still have lingering questions here or can this be closed?

Thanks, @nohe427

doneill commented 8 years ago

Thanks @nohe427, yes you can close as resolved. Thanks!