rudderlabs / rudder-sdk-android

Android SDK for RudderStack - the Customer Data Platform for Developers.
https://www.rudderstack.com
Other
18 stars 28 forks source link

fix: android sample application with rudderstack.properties #415

Closed ChryssaAliferi closed 3 months ago

ChryssaAliferi commented 3 months ago

Ticket

Description

In this PR, we are solving the problem that was caused in the previous merge to feat/android due to the missing property of instanceName that is now unused, but also we are improving the local.properties file setup, by creating a rudderstack.properties.sample (which needs to be replaced) and by guiding our clients to have the rudderstack.properties out of their Version Control Systems. Also, a minor refactoring change in the sample application's build.gradle.kts in order to portray a more granular error if the rudderstack.properties file is missing.

Type of change

Implementation Details

Checklist:

Screenshots

sonarcloud[bot] commented 3 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

saikumarrs commented 3 months ago

@ChryssaAliferi, maybe we should treat this as a "chore," as the changes only pertain to the sample apps and do not affect the functionality of the SDK itself.

ChryssaAliferi commented 3 months ago

@ChryssaAliferi, maybe we should treat this as a "chore," as the changes only pertain to the sample apps and do not affect the functionality of the SDK itself.

It crossed my mind, however, because as project this was not buildable, I preferred the fix tag.