Tipsi-stripe: Manifest error

[trustfall]

Followed instructions for adding Android, I get the following error when building in Android Studio:

Error:Execution failed for task ':app:processDebugManifest'.
> Manifest merger failed : Attribute data@scheme at AndroidManifest.xml requires a placeholder substitution but no value for <tipsiStripeRedirectScheme> is provided.

[trustfall]

Nevermind, further down in the docs

[isnifer]

I will reopen the issue, we need to update docs

[adlondon]

Also having this issue for android. How did you solve it @trustfall

[paynd]

Hi @adlondon.
Just add

manifestPlaceholders = [
            tipsiStripeRedirectScheme: "example"
        ]

into defaultConfig section of your app build.gradle
I'm going to update documentation, asap.

[paynd]

@adlondon Actually we alredy had this in out docs

[kylealwyn]

No matter where I put manifestPlaceholders in build.gradle, still getting this error. I've tried defaultConfig, buildTypes and productFlavors. Cleaned, rebuilt, uninstalled and reinstalled countless times

Current config:

    defaultConfig {
        applicationId "co.myapp"
        minSdkVersion 19
        targetSdkVersion 26
        versionCode 6
        versionName "1.2.1"
        multiDexEnabled true

        manifestPlaceholders = [
                tipsiStripeRedirectScheme: "scheeeme"
        ]

        ndk {
            abiFilters "armeabi-v7a", "x86"
        }
    }

Current Output:

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':tipsi-stripe:processDebugAndroidTestManifest'.
> Manifest merger failed : Attribute data@scheme at manifestMerger2118535579845422616.xml requires a placeholder substitution but no value for <tipsiStripeRedirectScheme> is provided.

Android Studio 3.0.1
Gradle 4.3

[trustfall]

I found for me it doesn't recognize the redirect scheme at all. If I use an ejected build of CRNA, it will build with the expokit scheme given, but only if I build directly to device or emulator. If I just rebuild, it will give the same error. Expokit is kind of a different animal, but maybe there's something there.

[kylealwyn]

Interesting, thanks for the followup. Not using CRNA or Expo here though. Downgraded to 4.1.2 in the meantime til I or someone smarter than me can figure it out

[trustfall]

worst case scenario, you can always hardcode the value in the tipsi manifest file.

[BearJS]

Exact similar issues @kylealwyn I have added the manifest placeholder however my stack trace shows the following issue:

Task :app:processDebugResources
Failed to execute aapt
com.android.ide.common.process.ProcessException: Failed to execute aapt

[isnifer]

@BearJS https://github.com/tipsi/tipsi-stripe/blob/master/example/android/gradle.properties#L21

[BearJS]

@isnifer the value for these properties are already correct

[adlondon]

Not sure if it helps anyone here, but I was still having this issue running an outdated version of NPM. Don't know how it got that way, but updating to NPM @5.5.1 and Node @8.9.3 set me right

[paynd]

@BearJS can you post buildToolsVersion, compileSdkVersion, defaultConfig { ... } and dependencies { ... } sections from your application level build.gradle?
@trustfall @kylealwyn can you try #205 ?

[kingarawana]

I'm having the same issue.

`Execution failed for task ':tipsi-stripe:processDebugAndroidTestManifest'.

Manifest merger failed : Attribute data@scheme at manifestMerger1426022766957738802.xml requires a placeholder substitution but no value for is provided.`

I tried doing as the docs suggested and added it under android.defaultConfig.manifestPlaceholders but it failed. I also tried adding it under buildTypes.debug.defaultConfig.manifestPlaceholders since it said "processDebug.." but that also didn't help.

Here are my current project configs

applicationId "com.myapp" minSdkVersion 17 targetSdkVersion 22 versionCode 9 versionName "1.5.4" multiDexEnabled true

I also have several several product flavors. Not sure if that would have anything to do with it.

[paynd]

@kingarawana try to use this version:

"dependencies": {
    ...
    "tipsi-stripe": "[email protected]:tipsi/tipsi-stripe.git#1e513b7"
  },

There are #205 .
If it does not help post your build.gradle.

[isnifer]

Fixed in 4.5.4 tipsiStripeRedirectScheme is not required anymore